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

Update Clapper icon #3810

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

Conversation

Raindrac
Copy link

@Raindrac Raindrac commented Aug 31, 2024

This commit remakes the Clapper icon from the ground up to fix various problems that I found, such as misaligned nodes. All six sizes were remade, colors were adjusted, and it's overall more in-line with Papirus's style. It took around two hours.

Design Rationale

  • The hinge was made a lighter color to better compliment the color of the stripes.
  • I removed the shadow from the hinge and instead gave it a highlight to better match the Papirus style. Papirus doesn't usually have shadows on the left side of an object, as they're usually drop shadows that are directly below it instead. Although that was faithful to the shading of the original, I found that it caused it to look out of place next to other Papirus icons.
  • The leftmost stripe on the top stick was extended so that it seamlessly connects to the hinge. This gets rid of the awkwardly thin stripe of blue that was visible right next to the hinge and less than a pixel wide.
  • Proportions and alignments were made more accurate to expectations and misaligned nodes were fixed. The clapper was slightly widened to create a square bounding box. A third stripe was also added to each stick for better symmetry.

@morganist
Copy link
Contributor

I like it, it looks better and is lighter in filesize, but again there is a lot of transform. You could search for a solution to get rid of it on the web, but i found out that exporting as a "plain SVG" in Inkscape could work. If that doesn't get rid of transform, try avoid copy-pasting elements between files.

@Raindrac
Copy link
Author

Removed all of the transform tags and optimized the svg. It should be ready to go

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