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

Change Square-1 yellow to dark gray #50

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

Conversation

alexmaasswca
Copy link

@alexmaasswca alexmaasswca commented Mar 18, 2024

Changed Square-1 yellow to dark gray by adding a new color, this is the most common square-1 color scheme, and fixes the problem with yellow and white contrasting, this would make it easier to check scrambles, and also make it easier on our eyes. The community has already voiced support for this change. https://forum.worldcubeassociation.org/t/change-clock-and-square-1-color-scheme-on-tnoodle-scramble-pdfs-to-white-black/42647

Obviously I know you can change it manually, but I think it would be good to change the default for the reasons above.

Before:
image

After:
image

Changed Square-1 yellow to dark gray by adding a new color, this is the most common square-1 color scheme, and fixes the problem with yellow and white contrasting, this would make it easier to check scrambles, and also make it easier on our eyes. The community has already voiced support for this change. https://forum.worldcubeassociation.org/t/change-clock-and-square-1-color-scheme-on-tnoodle-scramble-pdfs-to-white-black/42647
@gregorbg
Copy link
Member

Thank you for your submission, it is a reasonable proposal! The code looks fine, but I am holding off the merge because it would require me to compile new releases for all current TNoodle binaries (which is something I'm not willing to do with potential replacement solutions on the horizon)

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