Skip to content
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

Graphic.js: darker green for qsubcolor1 #280

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

glasser
Copy link
Contributor

@glasser glasser commented Nov 18, 2022

This makes the green color used for qsubcolor1 (eg, for slither) darker and thus more distinguishable from the yellow used for qsubcolor2 by people with some forms of colorblindness.

The particular color chosen is inspired by https://sashamaps.net/docs/resources/20-colors/ though I am not tweaking the other colors to come from here.

Fixes #255.

This makes the green color used for qsubcolor1 (eg, for slither) darker and thus more distinguishable from the yellow used for qsubcolor2 by people with some forms of colorblindness.

The particular color chosen is inspired by https://sashamaps.net/docs/resources/20-colors/ though I am not tweaking the other colors to come from here.

Fixes robx#255.
@vercel
Copy link

vercel bot commented Nov 18, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
pzprjs ✅ Ready (Inspect) Visit Preview Nov 18, 2022 at 0:21AM (UTC)

@glasser
Copy link
Contributor Author

glasser commented Nov 18, 2022

Hmm, this is probably too close to the default (not "color each line") line color:

image

@codecov-commenter
Copy link

Codecov Report

Base: 88.58% // Head: 88.58% // No change to project coverage 👍

Coverage data is based on head (a0528cb) compared to base (f666ceb).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #280   +/-   ##
=======================================
  Coverage   88.58%   88.58%           
=======================================
  Files         164      164           
  Lines       17927    17927           
=======================================
  Hits        15880    15880           
  Misses       2047     2047           
Impacted Files Coverage Δ
src/puzzle/Graphic.js 94.91% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@robx
Copy link
Owner

robx commented Nov 18, 2022

Yeah I don't think we should go for darker shading. I was hoping (one of) the shades could be tweaked to something that's more apparently different from the other, without increasing the "contrast" in terms of darkness.

@glasser
Copy link
Contributor Author

glasser commented Nov 18, 2022

Well, the contrast in terms of darkness is admittedly what many colorblind folks can see — but if you're comfortable just completely changing a color then that works too. I use pink as my secondary color in penpa fwiw.

@robx
Copy link
Owner

robx commented Nov 18, 2022

So what would be ideal is to have a change that keeps looking "nice" for everyone but is better in terms of accessibility, and particularly good enough for you. I'm not opposed to switching out the colors within those parameters.

I don't know enough about the various forms of color blindness to be able to tell whether it's realistic to find a single color scheme that works, there.

If that doesn't work, I think the best alternative would be to add some kind of accessibility toggle (whether that be via stylesheet or a menu option) which would switch out the color scheme, and then you can go wild with how you'd like puzzles to look in "accessible mode". (Needn't be a single option if that's not selective enough, but it would be a good start.) That needn't be limited to changing the colors, e.g. it would be possible to draw thin XOs for the two shadings.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Slitherlink colors are challenging for my colorblindness
3 participants