primer / octicons

A scalable set of icons handcrafted with <3 by GitHub
https://primer.style/foundations/icons
MIT License
8.32k stars 827 forks source link

[Feedback] Consider a filled icon treatment for hollow icons #840

Closed ericwbailey closed 2 years ago

ericwbailey commented 2 years ago

Note: Moved to https://github.com/github/primer/issues/1303

Describe the topic

This issue arose from a conversation in https://github.com/github/accessibility-audits/issues/477.

The gist is that we are currently relying on only color to communicate state, which runs afoul of WCAG success criterion Understanding Success Criterion 1.4.1: Use of Color.

One proposal for addressing the issue was to use a filled treatment for the icons.

Example from Reddit's up/downvote functionality, as provided by @stephaniegiang:

Two arrows, one pointing up and one pointing down. The up arrow has a filled color, indicating someone has upvoted. There are  370 votes present. Screenshot.

This solution is ideal in that it provides a visually obvious indication of a pressed/active state. Compare this to a bolded text treatment of the vote count, which:

Additionally, a filled treatment significantly reduces the complexity of managing color contrast requirements for state.

While filled arrows would help with this issue in particular, I feel filled treatments would also be useful to have for many Octicons for similar usability issues, both current and future.

ericwbailey commented 2 years ago

Closing to move the issue over to https://github.com/github/primer/issues/1303.