numix-core icon indicating copy to clipboard operation
numix-core copied to clipboard

Removed problematic system-* icons

Open Foggalong opened this issue 7 years ago • 10 comments

Fixes #3681

Foggalong avatar Sep 19 '18 17:09 Foggalong

Will have a look in 2 hours or so.

palob avatar Sep 19 '18 17:09 palob

I wasn't so sure on gnome-session-hibernate and gnome-session-suspend. Does anyone who uses GNOME know if those are used exclusively in launchers or are they action icons also?

Foggalong avatar Sep 19 '18 17:09 Foggalong

AFAIK Ubuntu uses/used such launchers but I think the icons are named differently. I can't see them pop up in my GNOME. Since GNOME switched pretty much everything except for app icons to symbolic I can't think of a place where they would. Even Adwaita doesn't contain any session-*.

Given that we've got mate-session-* icons in our theme maybe MATE, Xfce or Cinnamon do use this type of icons

palob avatar Sep 19 '18 18:09 palob

Found the icons in the app directory of the Linux mint theme

Same for elementary-xfce.

palob avatar Sep 19 '18 19:09 palob

No such app icons in the MATE theme https://github.com/mate-desktop/mate-icon-theme.

palob avatar Sep 19 '18 19:09 palob

Xfce even got the removed ones :-( so we'll probably have to check some distros in real life.

palob avatar Sep 19 '18 19:09 palob

Damn, will asign this to next release so we can do some proper testing.

Foggalong avatar Sep 19 '18 19:09 Foggalong

Those icons should be removed and moved to Numix base theme as they are actions and not apps. That's where they are installed on the elementary theme (they follow the standards more than XFCE/Mate guys.) Those GNOME icons are from GNOME2. They are not found here on my system (Fedora Rawhide with GNOME)

bilelmoussaoui avatar Jan 06 '19 12:01 bilelmoussaoui

@bilelmoussaoui It might be the case, but we do need to make sure removing them outright won't break things

Foggalong avatar Jan 29 '19 15:01 Foggalong

Found the icons in the app directory of the Linux mint theme

Same for elementary-xfce.

Looks like this isn't the case anymore.

palob avatar Oct 28 '24 00:10 palob