Skip to content

Conversation

@thatblindgeye
Copy link
Contributor

What: Closes #7022

I also removed thearia-labels from various examples except for the standalone example (as that one was the only one that needs it). Otherwise screen readers wouldn't announce the label for the checkbox.

Additional issues:

@patternfly-build
Copy link
Collaborator

patternfly-build commented Mar 9, 2022

Copy link
Contributor

@tlabaj tlabaj left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@jessiehuff jessiehuff left a comment

Choose a reason for hiding this comment

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

LGTM!

@tlabaj tlabaj merged commit 6a72fe7 into patternfly:main Mar 16, 2022
@patternfly-build
Copy link
Collaborator

Your changes have been released in:

  • eslint-plugin-patternfly-react@4.38.1
  • @patternfly/react-catalog-view-extension@4.50.1
  • @patternfly/react-charts@6.52.1
  • @patternfly/react-code-editor@4.40.1
  • @patternfly/react-console@4.50.1
  • @patternfly/react-core@4.199.1
  • @patternfly/react-docs@5.60.1
  • @patternfly/react-icons@4.50.1
  • @patternfly/react-inline-edit-extension@4.44.1
  • demo-app-ts@4.159.1
  • @patternfly/react-integration@4.161.1
  • @patternfly/react-log-viewer@4.44.1
  • @patternfly/react-styles@4.49.1
  • @patternfly/react-table@4.68.1
  • @patternfly/react-tokens@4.51.1
  • @patternfly/react-topology@4.46.1
  • @patternfly/react-virtualized-extension@4.46.1
  • transformer-cjs-imports@4.37.1

Thanks for your contribution! 🎉

@thatblindgeye thatblindgeye deleted the checkbox_conversion branch March 21, 2022 12:36
wise-king-sullyman pushed a commit to wise-king-sullyman/patternfly-react that referenced this pull request Mar 23, 2022
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.

Checkbox: convert examples to TypeScript

5 participants