Skip to content

A lot of new icons and some minor changes#208

Merged
jfly merged 3 commits intocubing:mainfrom
kognise:main
Feb 27, 2026
Merged

A lot of new icons and some minor changes#208
jfly merged 3 commits intocubing:mainfrom
kognise:main

Conversation

@kognise
Copy link
Contributor

@kognise kognise commented Feb 27, 2026

Hey! I made a few icons that I would love to contribute back to this repo. Happy to make any changes if needed :)

If merged, this should supersede #204 and #205.

New icons

I added the following puzzles:

  • Cmetrick Mini
  • Crazy 3x3x3
  • Dino Cube
  • Gear Cube
  • Gigaminx
  • Corner Helicopter 2x2x2
  • Ivy Cube
  • Pyraminx Crystal
  • Square-2
  • Super Square-1
  • UFO

All new icons are in line with existing icon styles, optimized with SVGO, of a relatively small file size, and verified to render correctly on a local build of the website. It's probably easier to open the PR changes tab for previews of the icons than for me to insert them all into this description.

Modified icons

I also made a few extremely minor changes to existing icons:

Megaminx

Fixes a weird inconsistency where in the old icon, a few corners were randomly rounded but not in any consistent fashion. Otherwise, the new icon is visually indistinguishable from the old one. The new icon file is also a lot smaller.

Square-1

Fixes a bug where the old icon didn't have a viewbox specified, causing rendering to fail in some cases. The lines crossing the icon also now properly divide the cube (the old icon wouldn't have been turnable). I made sure to keep the visual weight the same.

3x3x3 Mirror Blocks Blind

This was the only icon that has a fill set to a fixed color (#000) rather than inherited from the parent, so I fixed this. Otherwise, the actual contents of the icon file is untouched.

@lgarron
Copy link
Member

lgarron commented Feb 27, 2026

Thanks, I won't be able to review them in detail right now but they look quite good!

We certainly had a bunch of inconsistencies that I've been meaning to clean up; thanks for tackling some of them.

@lgarron
Copy link
Member

lgarron commented Feb 27, 2026

This was the only icon that has a fill set to a fixed color (#000) rather than inherited from the parent, so I fixed this. Otherwise, the actual contents of the icon file is untouched.

Also, if you feel up for it I think it would be good to add a regression test for this (by checking the styles and fill attributes of all elements) in a followup PR (which also will require fixing 333_oven_mitts).

@kognise
Copy link
Contributor Author

kognise commented Feb 27, 2026

oh wow 333_oven_mitts is fully inkscaped up 😭

i'll take a look at doing so if i have some extra time!

Copy link
Member

@jfly jfly left a comment

Choose a reason for hiding this comment

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

Great work, thank you for the high quality contribution, @kognise!

Do you expect to make more changed to this PR, or will subsequent changes all be in #209? If this is done, I will merge it.

@kognise
Copy link
Contributor Author

kognise commented Feb 27, 2026

Thanks for the stamp! No more changes in this PR, everything else will be separate.

@jfly jfly merged commit e23c21a into cubing:main Feb 27, 2026
2 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.

3 participants