Skip to content

Add icon for KDE Partition Manager and GNOME Disks fixes - #3505

Merged
SmartFinn merged 5 commits into
PapirusDevelopmentTeam:masterfrom
morganist:partitionmanager
Aug 2, 2023
Merged

Add icon for KDE Partition Manager and GNOME Disks fixes#3505
SmartFinn merged 5 commits into
PapirusDevelopmentTeam:masterfrom
morganist:partitionmanager

Conversation

@morganist

Copy link
Copy Markdown
Member

closes #3451
Tweaked the shadows for org.gnome.DiskUtility;
Remade the 32x32 org.gnome.DiskUtility.svg to make it pixel-perfect;
Remade the 16x16 org.gnome.DiskUtility.svg to make it pixel-perfect, less busy, simpler, and more readable;
Added an icon for KDE Partition Manager to the file partitionmanager.svg;
Added a symlink org.kde.partitionmanager.svg and symlinked it to org.kde.partitionmanager;

image
Previous GNOME Disks 16x16 icon and a new one side by side

image
New icon for KDE Partition manager and an original one

@SmartFinn

Copy link
Copy Markdown
Member

@morganist Thanks for the icons.

Could you try to improve the pixel alignment in 48px and 64px icon?

image

@SmartFinn

Copy link
Copy Markdown
Member

@morganist I forget to say that 22px and 24px are already fixed locally.

@morganist

Copy link
Copy Markdown
Member Author

@morganist I forget to say that 22px and 24px are already fixed locally.

Oh, okay. So you'll push these fixes after merging this pull request?

@SmartFinn

Copy link
Copy Markdown
Member

@morganist no, commit the fixes here now

@morganist

Copy link
Copy Markdown
Member Author

I made 48x48 and 64x64 icons pixel-perfect (for both partitionmanager and org.gnome.DiskUtility)
And you do already have a fix for 22x22 and 24x24 icons locally, right?

@SmartFinn

Copy link
Copy Markdown
Member

@morganist

I made 48x48 and 64x64 icons pixel-perfect (for both partitionmanager and org.gnome.DiskUtility)

Nice work. Thank you!

And you do already have a fix for 22x22 and 24x24 icons locally, right?

This is correct.

- Fix render issue with the ring
- Remove outline from disk icons
@SmartFinn
SmartFinn merged commit f3ddb0a into PapirusDevelopmentTeam:master Aug 2, 2023
@morganist
morganist deleted the partitionmanager branch August 2, 2023 19:07
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.

[Icon request] KDE Partition Manager

2 participants