Skip to content

Polish Common Lisp mimetype icons#3696

Closed
UM-Li wants to merge 3 commits intoPapirusDevelopmentTeam:masterfrom
UM-Li:master
Closed

Polish Common Lisp mimetype icons#3696
UM-Li wants to merge 3 commits intoPapirusDevelopmentTeam:masterfrom
UM-Li:master

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>
@SmartFinn SmartFinn closed this Apr 29, 2025
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.

4 participants