-
Notifications
You must be signed in to change notification settings - Fork 24
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
Add Patterns and Textures to Bar Graphs #3146
Comments
@SamiyahKey it seems like this should probably be a design ticket first to determine the patterns, textures or other non-color indicators that dev should implement. Are the items in Additional Context actually acceptance criteria? They read like additional requirements, but it's not clear to me. |
@katypies I was thinking the same thing until I discussed it with @Katherine-Osos. Because the Django admin is pretty limited with design options, this would skip design, and dev should partner with design (me) to use what's available in the most effective and accessible way possible. In terms of the Additional Context, I attempted to point out that the legend should be updated to reflect the new colors/patterns chosen. I changed them to requirements! Thanks for that catch. |
Note: There is a chart.js library aimed at adding this exact thing, with the purpose of resolving color blindness issues. Might be worth looking at |
#3146: Django admin analytics page - add patterns to our bar graphs for colorblind users - [ZA]
Story
As a with visual impairments or color vision deficiencies,
I need bar graphs to include patterns or textures in addition to color to differentiate data points,
so that I can accurately interpret the data being presented without relying on color alone.
https://getgov-staging.app.cloud.gov/admin/analytics/#content-start
Acceptance Criteria
Additional Context
Issue Links
Implementation notes:
@zandercymatics did some research and found an add-on for the JS charting library we're using (link). Alternatively, consider using a sprite and CSS background.
The text was updated successfully, but these errors were encountered: