-
Notifications
You must be signed in to change notification settings - Fork 147
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
Removed problematic system-* icons #4585
base: master
Are you sure you want to change the base?
Conversation
Will have a look in 2 hours or so. |
I wasn't so sure on |
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. Given that we've got |
Found the icons in the Same for elementary-xfce. |
No such app icons in the MATE theme https://github.com/mate-desktop/mate-icon-theme. |
Xfce even got the removed ones :-( so we'll probably have to check some distros in real life. |
Damn, will asign this to next release so we can do some proper testing. |
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.) |
@bilelmoussaoui It might be the case, but we do need to make sure removing them outright won't break things |
Looks like this isn't the case anymore. |
Fixes #3681