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

[Icon request] KDE Partition Manager #3451

Closed
ocobblepot opened this issue Jun 12, 2023 · 18 comments · Fixed by #3505
Closed

[Icon request] KDE Partition Manager #3451

ocobblepot opened this issue Jun 12, 2023 · 18 comments · Fixed by #3505

Comments

@ocobblepot
Copy link

KDE has updated the icon for its Partition Manager.

The icon name in Papirus (and Breeze) is partitionmanager.svg

The new icon is here https://github.com/KDE/breeze-icons/blob/master/icons/apps/48/partitionmanager.svg

@morganist
Copy link
Contributor

morganist commented Jul 30, 2023

@ocobblepot
But partitionmanager.svg is a standard icon for all partition managers in Papirus. Do we really need a special icon for a partition manager that is made by KDE?

@ocobblepot
Copy link
Author

ocobblepot commented Jul 30, 2023

But partitionmanager.svg is a standard icon for all partition managers in Papirus.

I couldn't find any other app icons with "part" in the name that link to it.

Do we really need a special icon for a partition manager that is made by KDE?

GParted has it's own icon (gparted.svg). Also the current Papirus icon for partitionmanager.svg resembles the Breeze icon for Filelight https://github.com/KDE/breeze-icons/blob/master/icons/apps/48/filelight.svg rather than the new one for Partition manager above.

@morganist
Copy link
Contributor

morganist commented Jul 30, 2023

so, we could symlink filelight papirus icon to partitionmanager.svg because of the resamblance. But:
@SmartFinn categorised disk icons here back in 2017: 5fa399a
Since then, many disk apps gained their own icon (for example, baobab)
Maybe we need to rename partitionmanager.svg to disk-usage-analyzer.svg? And vise versa? Or maybe we also need to set the gparted icon as a standard partition manager icon and use it for this one too...

@morganist
Copy link
Contributor

Actually, now i agree that KDE partition manager needs its own icon – there isn't a lot of GUI partition managers availible on linux, it seems. But the filelight icon issue still persists. Also, is there any app symlinks in Papirus that use partitionmanager.svg? If not, would it really be better to use its icon for filefight and other disk usage analyzers, and make a new one for it, for KDE partition manager? After all, KDE literally named it partitionmanager...

@morganist
Copy link
Contributor

Papirus partition manager icon and Filelight icon:

image

The partition manager icon really should be dedicated to KDE partition manager, while the current icon is a better fit for disk image analyzers.

@morganist
Copy link
Contributor

@SmartFinn do we need a separate issue for that?

@SmartFinn
Copy link
Member

@morganist that's make sense. I'll review the changes made in this commit 5fa399a

@morganist
Copy link
Contributor

morganist commented Aug 1, 2023

how about this design?

image

@SmartFinn
Copy link
Member

@morganist

how about this design?

This is a good one. I like it. This is may be just a circle for 16/22/24px icons, and a drive with the circle for 32/48/64px.

@morganist
Copy link
Contributor

morganist commented Aug 1, 2023

cool, i agree!
i'll try.
but we still need to solve the problem on how to handle disk-usage-analyzer and partitionmanager.
i think papirus doesn't need a general icon for partition managers, there's, like, only 4 of them (gparted, gnome disks, kde partition manager, yast-disk).
we just give them all a unique icon.
i'll do kde partition manager today, and something needs to be done with yast-disk (openSUSE's YaST Expert Partitioner)

@morganist
Copy link
Contributor

i think it'll be fine if yast-disk.svg stays symlinked to disk-utility.svg

@SmartFinn
Copy link
Member

@morganist

but we still need to solve the problem on how to handle disk-usage-analyzer and partitionmanager.

Just wait with the PR, I'll replace it today.

@SmartFinn
Copy link
Member

@morganist let's reschedule this changes to the next release. 6666 commits is a nice number to tag the release. I'll do the changes tomorrow, or today to a separated branch.

@morganist
Copy link
Contributor

let's reschedule this changes to the next release.

and when would you do a full release?

@SmartFinn
Copy link
Member

@morganist

and when would you do a full release?

It's today.

SmartFinn added a commit that referenced this issue Aug 2, 2023
Replace partitionmanager icon with a symlink to gparted (temporary)
@SmartFinn
Copy link
Member

@morganist

I decided, don't replace disk-usage-analyzer with partitionmanager icon because it's only filelight and mate-disk-usage-analyzer have the rings chart view from the whole list:

disk-usage-analyzer.svg
|- diskmonitor.svg
|- gdmap_icon.svg
|- gsmartcontrol.svg
|- kdf.svg
|- kdiskfree.svg
|- kwikdisk.svg
|- mate-disk-usage-analyzer.svg
|- org.kde.plasma.systemmonitor.diskactivity.svg
|- org.kde.plasma.systemmonitor.diskusage.svg
|- preferences-smart-status.svg
|- xfce-mount.svg

Now, the old partitionmanager icon is filelight icon. partitionmanager is a temporary symlink to gparted icon and wait for your PR.

@morganist
Copy link
Contributor

morganist commented Aug 2, 2023 via email

@ocobblepot
Copy link
Author

Thank you both.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants