-
Notifications
You must be signed in to change notification settings - Fork 46
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
[Twizzle issue] Icosahedron color scheme hurts to look at. #336
Comments
Could you confirm the colours that are meant to be opposite each other? Is white opposite purple-grey?? |
I'd definitely love to update the colors. Are you able to provide more context on the provided screenshot, or examples of it in action? I certainly like that (dark) green and white are in the traditional positions, but I'm a little concerned about three shades of lavender and there are fewer symmetries and similarities with other puzzles than possible. |
…stinct. This implements the colors from #336 (comment) with slight tweaks: - Lavender: slightly lighter for better contrast with other colors - Black: replaced with (very) dark gray to avoid blending with the foundation color.
…stinct. This implements the colors from #336 (comment) with slight tweaks: - Lavender: slightly lighter for better contrast with other colors - Black: replaced with (very) dark gray to avoid blending with the foundation color.
@Lord0fTheCubes Apologies for the delay, as you might imagine I've been rather occupied. Thanks for filing this issue, it made it easy for me to pick up the context again! I've implemented and deployed the following: Could I ask you to consult with other people who were interested in a color scheme change and let me know you and their feedback? Thanks! |
Thank you so much! I'm waiting to hear back from the icosahedron thread on Discord, but I love the new color scheme. Replacing black with charcoal was a good idea, to avoid it being the same color as the puzzle body. The only nitpick I have about this is that you used a darker shade of blue on the dodecahedron color scheme, and icosahedra would benefit from using that blue to avoid any possible confusion with the lighter shade blue. And while I'm on the topic, both DianSheng and DaYan used a dark gray on their FTOs rather than light gray, but the gray used in Twizzle is still light. Idk how hard it is to change color schemes, so I hope I'm not asking too much. Thanks for everything! |
Good idea, done!
Good point. I've introduced |
Both of the updated colors look good to me. Thank you so much for making them differentiable! |
Glad to hear it! Gonna close out this issue for now, feel free to file new issues if anything comes to mind. |
Steps to reproduce the issue
Open Twizzle and click on any icosahedral puzzle
Observed behaviour
The color scheme is horrific. There are many weird shades of blue, purple, pink, and green, most of which are extremely similar to one another, and staple colors of twisty puzzles (white, yellow, red, etc.) are completely missing. I conducted a series of polls on the FTO Discord server and we decided that this is the new unofficial standard color scheme for icosahedral puzzles. Please fix this. The current icosahedron color scheme is such an eyesore that it is genuinely the main reason why I don't solve icosahedral puzzles in Twizzle.
🖼 Screenshots
I cannot vibe with this.
Expected behaviour
I expected the icosahedron color scheme to look like literally anything other than it currently does (preferably the color scheme I linked above).
Browser & operating system
Chrome (idk what version) on Mac OS
Additional info
Here are some of the names for the icosahedral puzzles listed on the site:
Icosahedron 2 = Regular Astrominx
Icosahedron 3 = Regular Astrominx + Big Chop
Icosahedron moving faces = Icosaminx
Here are some more that you could include to broaden the default puzzle selection in Twizzle:
Potential puzzles that could be added (sorry I'm getting a little carried away with this whole icosahedron thing)
Radiolarian 2: i f 0.7453559925
Radiolarian 3: i f 0.7
Astrominx (not regular astrominx, just astrominx): i v 0.275
The text was updated successfully, but these errors were encountered: