Skip to content

fix(Calendar month) aria labels for dates outside of the selected month are inaccurate#7720

Merged
thatblindgeye merged 2 commits intopatternfly:mainfrom
dominik-petrik:CalendarMonth
Jul 25, 2022
Merged

fix(Calendar month) aria labels for dates outside of the selected month are inaccurate#7720
thatblindgeye merged 2 commits intopatternfly:mainfrom
dominik-petrik:CalendarMonth

Conversation

@dominik-petrik
Copy link
Copy Markdown
Contributor

What: Closes #7718

There was same issue with years in aria label. 31 December 2023 date had "31 January 2024" aria label. This should be also resolved by this pull request.

@dominik-petrik dominik-petrik changed the title Calendar month fix(Calendar month) aria labels for dates outside of the selected month are inaccurate Jul 20, 2022
@patternfly-build
Copy link
Copy Markdown
Collaborator

patternfly-build commented Jul 20, 2022

@nicolethoen
Copy link
Copy Markdown
Contributor

@dominik-petrik this PR needs to be rebased after your CalendarMonth PR gets merged before we can merge this - since you accidentally included your calendarmonth improvements in this PR.

Copy link
Copy Markdown
Contributor

@thatblindgeye thatblindgeye left a comment

Choose a reason for hiding this comment

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

The update for the aria-labels looks (and sounds) good! Once your other PR gets merged and this gets rebased like Nicole said I can approve 🙂

Copy link
Copy Markdown
Collaborator

@wise-king-sullyman wise-king-sullyman left a comment

Choose a reason for hiding this comment

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

This change looks good! Can you add a test for this as well?

@dominik-petrik
Copy link
Copy Markdown
Contributor Author

This change looks good! Can you add a test for this as well?

Sure thing!

Copy link
Copy Markdown
Collaborator

@wise-king-sullyman wise-king-sullyman left a comment

Choose a reason for hiding this comment

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

Looks great! Thanks for your quick work on this 🚀

@thatblindgeye thatblindgeye merged commit 338624b into patternfly:main Jul 25, 2022
@patternfly-build
Copy link
Copy Markdown
Collaborator

Your changes have been released in:

  • eslint-plugin-patternfly-react@4.64.10
  • @patternfly/react-catalog-view-extension@4.76.10
  • @patternfly/react-charts@6.78.10
  • @patternfly/react-code-editor@4.66.10
  • @patternfly/react-console@4.76.10
  • @patternfly/react-core@4.225.10
  • @patternfly/react-docs@5.86.10
  • @patternfly/react-icons@4.76.10
  • @patternfly/react-inline-edit-extension@4.70.10
  • demo-app-ts@4.185.10
  • @patternfly/react-integration@4.187.10
  • @patternfly/react-log-viewer@4.70.10
  • @patternfly/react-styles@4.75.10
  • @patternfly/react-table@4.94.10
  • @patternfly/react-tokens@4.77.10
  • @patternfly/react-topology@4.72.10
  • @patternfly/react-virtualized-extension@4.72.10
  • transformer-cjs-imports@4.63.10

Thanks for your contribution! 🎉

jenny-s51 pushed a commit to jenny-s51/patternfly-react that referenced this pull request Jul 26, 2022
…th are inaccurate (patternfly#7720)

* fix(CalendarMonth) add tests

* fix(CalendarMonth) update tests
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.

Bug - Calendar month - aria labels for dates outside of the selected month are inaccurate

5 participants