feat: improve accessibility for course progress bars#309
feat: improve accessibility for course progress bars#309singhankit001 wants to merge 1 commit intoOpenSource-Communities:mainfrom
Conversation
✅ Deploy Preview for learn-open-source ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
There was a problem hiding this comment.
PR Compliance Checks
Thank you for your Pull Request! We have run several checks on this pull request in order to make sure it's suitable for merging into this project. The results are listed in the following section.
Issue Reference
In order to be considered for merging, the pull request description must refer to a specific issue number.
This check is looking for a phrase similar to: "Fixes #XYZ" or "Resolves #XYZ" where XYZ is the issue number that this pull request is meant to address.
There was a problem hiding this comment.
Congratulations on submitting your first Pull Request! 🎉
We appreciate your contribution. Feel free to join the conversation in our
Community and say hello!
|
Hi @singhankit001. I noticed that your PR doesn't have an issue number, so I highly recommend closing this PR and reading the project's Contributing Guideto get a better idea on how to contribute to this project. That way, the maintainers can see how your idea can benefit the project. |
Added role="progressbar", aria-valuenow, aria-valuemin, and aria-valuemax to the course progress bars to improve accessibility for screen readers.