-
-
Notifications
You must be signed in to change notification settings - Fork 147
Accessible icon set #441
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Accessible icon set #441
Conversation
Adds the 'accessible' format, which replaces the "stoplight" colored circles, which may be unclear to colorblind users, with distinct icons, and replaces the up arrow, which uses a different visual language and may be described differently on screen readers, with one that stylistically matches the down arrow.
Coverage report
Test suite run success166 tests passing in 55 suites. Report generated by 🧪jest coverage report action from 3ca903e |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for pull request! This is really amazing change that I would love to have. Don't want to push it as default, would love to make it default in next major version
"small_red_triangle_down": "🔻", | ||
"green_circle": "✅", | ||
"yellow_circle": "⚠️", | ||
"red_circle": "❎", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't really like that it is green, maybe ⛔ or 🚫?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How about ❌ ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh, but there's also "x": "❌"
🤔
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just looking at how these are read by the screen reader in MacOS:
- "⛔" is read as "no entry"
- "🚫" is read as "restricted symbol"
- "❌" is read as "cross mark"
What do you think?
Some other options:
- "👍" for "green circle" and "👎" for "red circle" (keeping "
⚠️ " for "yellow circle") - "😁" for "green circle," "😬" for "yellow circle," and "😩" for "red circle"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Love the 👍/👎 variant
(ignore the CI, needs an update) |
The chick was read as "recently hatched chick still in shell," which is verbose; the replacement is read as 'new sign.'
Adds the 'accessible' format, which replaces the "stoplight" colored circles, which may be unclear to colorblind users, with distinct icons, and replaces the up arrow, which uses a different visual language and may be described differently on screen readers, with one that stylistically matches the down arrow.