Skip to content

Fix Sokoban's Colorblind Accessibility - #2943

Merged
manuq merged 1 commit into
endlessm:mainfrom
bergiebird:sokoban_colorblind_fix
Sep 22, 2026
Merged

manuq merged 1 commit into
endlessm:mainfrom
bergiebird:sokoban_colorblind_fix

Conversation

@bergiebird

@bergiebird bergiebird commented Sep 21, 2026

Copy link
Copy Markdown
Contributor

Changes:

  • Strengthened the thread pattern on spool_red.png & thread_red.png
  • Removed the thread pattern from spool_yellow.png & thread_yellow.png
  • Left green's pattern as-is

Results: Each color now has a distinct texture [smooth, medium, heavy], so spools and threads have an additional dimension to be more reliably told apart. Notes:

  • I am red/green colorblind. I'd say that it is now easier for me to see the dif between the different spools/threads.
  • No new colors were added to the color palette.
  • Deviates slightly from artist intent since the spools orignally shared a uniform thread pattern. Happy to readjust to better fit artist intent!

Resolves #1203

Changes:
- Strengthened the thread pattern on spool_red.png & thread_red.png
- Removed the thread pattern from spool_yellow.png & thread_yellow.png
- Left green's pattern as-is

Results: Each color now has a distinct texture [smooth, medium, heavy], so spools and threads have an additional dimension to be more reliably told apart.
Notes:
- I am red/green colorblind. I'd say that it is now easier for me to see the dif between the different spools/threads.
- No new colors were added to the color palette.
- Deviates slightly from artist intent since the spools orignally shared a uniform thread pattern. Happy to readjust to better fit artist intent!

@manuq manuq left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bergiebird thanks so much for your contribution! And you are welcome to open a bug or improve any other colordblind accessibility issue you see in the game.

Next time, please add a line in the pull request description to link it with the issue. Like "Resolves #1203". This will tie them and when the PR is merged, the issue will be marked as completed. Check https://github.com/endlessm/threadbare/wiki/Contributing#pull-request-descriptions

This time, I will add that line myself and merge. Thanks!

@manuq
manuq merged commit afa889f into endlessm:main Sep 22, 2026
6 checks passed
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.

Sokoban inaccessible to people with colorblindness

2 participants