Skip to content

fix result unreadable for colourblind #31

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

jasger9000
Copy link
Contributor

Hello again,

This PR is meant to fix the result page being hard/impossible to use for colour-blind people because the emojis are difficult to differentiate.

Since I'm not colour-blind myself, it would be really great if @q00u would have a look at it since they raised the issue initially from what I can tell. Any other feedback would also be greatly appreciated.

Before

before

Copied

Spellcheck #21
🟩🟩🟩🟩🟩
🟥🟩🟩🟥🟩
🟩🟩🟩🟥🟩

After

after

Copied

Spellcheck #21
✅✅✅✅✅
✅✅✅❌❌
✅✅✅✅❌

@q00u
Copy link

q00u commented Feb 2, 2024

Definitely readable and unambiguous no matter what kind of colorblindness (though I still like the softer blocks vs the checkmarks, if only it were red and blue)

In some cultures, the checkmark means wrong, and a circle is used for 'correct', but then again, this is about English, so maybe that doesn't matter

@jasger9000
Copy link
Contributor Author

though I still like the softer blocks vs the checkmarks

I could add blocks around the checkmarks and crosses, so the symbol would be black on a coloured cube.

if only it were red and blue

I think blue would be confusing for the majority of people, since green is usually associated with 'correct'

@jasger9000
Copy link
Contributor Author

I could add blocks around the checkmarks and crosses, so the symbol would be black on a coloured cube.

@q00u After a bit of messing with the width, this is what I came up with:

after

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.

2 participants