Clearer distinctions for pull request reviewer status and approvals #169699
Unanswered
ghostinhershell
asked this question in
Accessibility
Replies: 3 comments 2 replies
This comment was marked as off-topic.
This comment was marked as off-topic.
-
|
Somehow missed this discussion on my search... My feedback is here: https://github.com/orgs/community/discussions/169946 Basically, I think this has the potential to be a really good change but needs to be labelled clearer. |
Beta Was this translation helpful? Give feedback.
0 replies
-
|
This change affects the API webhook? because that thing send when a user approve but i think that not send if that approve counts or not like in the web. |
Beta Was this translation helpful? Give feedback.
2 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
We’ve enhanced accessibility in the pull request (PR) review process by making it easier for everyone to understand reviewer status without relying solely on color.
What’s changed
Previously, green and gray checkmarks were used to indicate whether a reviewer had write access or read-only/no access, but these colors were hard to distinguish and didn’t meet accessibility standards. The following updates are being released:
You can see an example of these changes in WordPress/wordpress-develop#2393 or in the screenshots below
Why does this matter?
By introducing explicit indicators and descriptive text, we make the PR review process more inclusive for everyone. This change was shaped with additional input from maintainers who helped us rethink what information maintainers and contributors actually need: a clear signal about the PR’s progress and reviewer status, not just visual cues that are easy to miss.
Feedback
We’re always looking to improve our usability and your feedback helps us get there! If you have suggestions or run into issues, leave a comment below.
Beta Was this translation helpful? Give feedback.
All reactions