-
Notifications
You must be signed in to change notification settings - Fork 869
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
Add keyboard accessibility #757
base: master
Are you sure you want to change the base?
Conversation
These are the things from the original issue which I didn't add in this PR:
|
CI build failed due to these reasons:
|
Hey @vahissan ! Thanks for this great PR, I need to find some time to review it :) If you know what's failing with the v7 of npm, can create a different PR with the fix and we merge it before this one? |
Reason 1 is resolved. But CI still fails due to reason 2. Looks like CI does not create a new build before running unit tests. |
Hey @arqex ! I know you must be busy. Just wanted to send a reminder in case you forgot. |
Hey @arqex |
Description
Motivation and Context
Fixes #415 (at least partially)
I was using the library and would like to have the ability to operate the date picker without a mouse. I'm sure many others would like it too.
Checklist