Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve Material Time Pickers #1792

Merged
merged 1 commit into from
Jul 30, 2021

Conversation

sdirix
Copy link
Member

@sdirix sdirix commented Jul 30, 2021

Reworks date, time and date-time support in the React Material
renderer set. The controls now support many customization options
and can be invoked separately via the ui schema.

We now use the much smaller 'dayjs' library (compared to 'moment')
for Javascript date support. It can be configured globally and is
therefore easily customizable on client side for their locales.

Also streamlines the label prop which was only supporting string for
a long time now.

@sdirix sdirix requested a review from eneufeld July 30, 2021 07:56
Copy link
Member

@eneufeld eneufeld left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good to me, thank you!

@sdirix sdirix marked this pull request as ready for review July 30, 2021 12:22
Reworks date, time and date-time support in the React Material
renderer set. The controls now support many customization options
and can be invoked separately via the ui schema.

We now use the much smaller 'dayjs' library (compared to 'moment')
for Javascript date support. It can be configured globally and is
therefore easily customizable on client side for their locales.

The date, time and date-time controls now use a similar layout
structure as normal controls, leading to a unified layout.

Also streamlines the label prop which was only supporting string for
a long time now.
@coveralls
Copy link

coveralls commented Jul 30, 2021

Coverage Status

Coverage increased (+0.1%) to 88.786% when pulling d22b693 on sdirix:improve-material-pickers into d6edf4b on eclipsesource:master.

@sdirix sdirix merged commit fc2d67f into eclipsesource:master Jul 30, 2021
@sdirix sdirix deleted the improve-material-pickers branch July 30, 2021 14:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants