-
Notifications
You must be signed in to change notification settings - Fork 8
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
Remove clearable
from color and time pickers
#1965
Conversation
clearable
from time pickersclearable
from color and time pickers
6d1b5cc
to
f413748
Compare
f413748
to
1a04615
Compare
@andrearutrecht is this ready for review? |
@johnnyomair No, it is still set to draft |
Then the review requests should be removed. Draft + review request = "It can't be merged like this, but I want your opinion on it anyway" Draft + no review request = "I'm still working on it" |
Oh, thank you for that info! I did not know. |
The
clearable
prop is removed from all pickers inadmin/admin-date-time
and fromColorPicker
. All Pickers are nowclearable
by default and only notclearable
whenrequired
is set.PR Checklist
Screenshots of Time and Date Pickers in Storybook to show default `clearable` behaviour:
Screenshots of Time and Date Pickers in Storybook to show `required` behaviour:
Screenshots of `ColorPicker` stories: