Thanks to visit codestin.com
Credit goes to github.com

Skip to content

chore(badge): added screenreader to unread badges#5708

Merged
mcoker merged 1 commit intopatternfly:mainfrom
ashley-o0o:5230-screenread
Aug 1, 2023
Merged

chore(badge): added screenreader to unread badges#5708
mcoker merged 1 commit intopatternfly:mainfrom
ashley-o0o:5230-screenread

Conversation

@ashley-o0o
Copy link
Contributor

closes: #5230

@patternfly-build
Copy link
Collaborator

patternfly-build commented Jul 5, 2023

@mcoker mcoker requested a review from thatblindgeye July 5, 2023 20:10
Copy link
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.

This looks good! Only couple things are:

  • There are a couple of unrelated commits as part of the PR
  • I'd defer to @mcoker regarding the formatting of the handlebars. Other instances of the screen-reader handlebars as it on a separate line from other content, but it might make more sense in those contexts

Copy link
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.

🎉 Thanks for making the update!

@mcoker mcoker merged commit 8651723 into patternfly:main Aug 1, 2023
@patternfly-build
Copy link
Collaborator

🎉 This PR is included in version 5.1.0-prerelease.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

@patternfly-build
Copy link
Collaborator

🎉 This PR is included in version 6.0.0-alpha.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Badge - Update examples to add screenreader text

4 participants