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

Adds keyboard navigation for month picker #2389

Merged
merged 8 commits into from
Sep 14, 2020

Conversation

camilaazuma
Copy link
Contributor

This PR:

@camilaazuma camilaazuma changed the title Fix/acessibility month Adds keyboard navigation for month picker Sep 11, 2020
@codecov
Copy link

codecov bot commented Sep 11, 2020

Codecov Report

Merging #2389 into master will increase coverage by 0.18%.
The diff coverage is 96.77%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2389      +/-   ##
==========================================
+ Coverage   92.08%   92.26%   +0.18%     
==========================================
  Files          20       20              
  Lines        1250     1280      +30     
  Branches      328      340      +12     
==========================================
+ Hits         1151     1181      +30     
  Misses         40       40              
  Partials       59       59              
Impacted Files Coverage Δ
src/calendar.jsx 93.69% <ø> (ø)
src/month.jsx 95.61% <96.77%> (+1.56%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dac2214...90c90cd. Read the comment docs.

@camilaazuma camilaazuma force-pushed the fix/acessibility-month branch from b4c1157 to ba00418 Compare September 11, 2020 22:26
Co-authored-by: Jonas Antonelli <[email protected]>
@martijnrusschen
Copy link
Member

Woohoo, nice work! Thanks for creating this PR!

@martijnrusschen martijnrusschen merged commit 73faa21 into Hacker0x01:master Sep 14, 2020
p-rogas pushed a commit to p-rogas/react-datepicker that referenced this pull request Mar 22, 2023
* feat: adds basic keyboard navigation to month picker

* fix: set focus when navigating through keyboard

* feat: add aria label to months

* fix: prevent navigation to disabed/excluded months

* tests: Tests added for new Month Picker feature

* docs: updates readme with new keybinds

* test: test coverage improved

* Update src/month.jsx

Co-authored-by: Jonas Antonelli <[email protected]>

Co-authored-by: Jonas Antonelli <[email protected]>
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.

Month picker navigation using keyboard keys not working
3 participants