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

Polish Common Lisp mimetype icons #3696

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

Conversation

UM-Li
Copy link
Contributor

@UM-Li UM-Li commented Apr 17, 2024

Redrew the yin-yang symbol with circles, so that the middle curve does not form angles with the outer ring.

1

@UM-Li
Copy link
Contributor Author

UM-Li commented Apr 17, 2024

Updated to make sure the light areas are of color #e4e4e4.

@morganist
Copy link
Member

hi,
your fixed icons have a bit of a problem - they use transform tags for some elements, which should be avoided.

@morganist
Copy link
Member

morganist commented Oct 12, 2024

try these steps to get rid of transform:

  • avoid putting elements in groups
  • be careful when copying-and-pasting elements between icons of different sizes
  • try to find the solution on the internet if these steps don't help

@achadwick
Copy link
Contributor

Doing this now. Circles are pain in the backside to remove transforms on (or maybe it's just rotations). Normally you can group + deep-ungroup, and the transforms will vanish. For these:

  1. Copy the object you need to nuke the rotation on
  2. On the copy, visit it in the XML editor Ctrl+Shift+X and remove its transform attribute.
    • It'll instantly teleport somewhere. Go and find it.
  3. Make sure you have snap to grids off (# normally does it)
  4. Set the colour of the original to something distinctive
  5. Move the copy back over the original, snapping to the original so the position is identical
  6. Bury the copy once by pressing PgDn
  7. Remove the distinctively coloured original

What a pain

achadwick added a commit to achadwick/papirus-icon-theme that referenced this pull request Jan 29, 2025
@achadwick
Copy link
Contributor

@morganist
Please can you merge https://github.com/achadwick/papirus-icon-theme/tree/pr3696-fixups to this? No transforms now.

@achadwick achadwick mentioned this pull request Jan 31, 2025
morganist pushed a commit that referenced this pull request Feb 6, 2025
* Polish Common Lisp mimetype icons

* Fix color

* Common Lisp mimetype: remove transforms.

updates #3696

---------

Co-authored-by: UM-Li <um-li@tuta.io>
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