-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
fix(DatePicker): support focus into a day with min/max #4793
fix(DatePicker): support focus into a day with min/max #4793
Conversation
This change allows user to focus a date into down-arrow key when min/max dates are enabled. When min/max dates are enabled, there can be no selected date in the calendar dropdown or calendar dropdown may not show today, so we need to look for another date to focus on. This change uses the first focusable date. Fixes carbon-design-system#4207.
Deploy preview for the-carbon-components ready! Built with commit 079b0eb https://deploy-preview-4793--the-carbon-components.netlify.com |
Deploy preview for carbon-components-react failed. Built with commit 079b0eb https://app.netlify.com/sites/carbon-components-react/deploys/5deeb1361d4c41000af6f6f8 |
Deploy preview for carbon-elements ready! Built with commit 079b0eb |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This change allows user to focus a date into down-arrow key when min/max dates are enabled. When min/max dates are enabled, there can be no selected date in the calendar dropdown or calendar dropdown may not show today, so we need to look for another date to focus on. This change uses the first focusable date.
Fixes #4207.
Changelog
New
Testing / Reviewing
Testing should make sure React date picker is not broken.