Manuel-Kehl / elementaryPlus

This theme is an addition to elementary OS's default icon theme. It contains icons for several third party applications that resemble elementary OS's style.
GNU General Public License v3.0
356 stars 35 forks source link

Installation should be installed inside the elementary icon theme #375

Closed Philip-Scott closed 8 years ago

Philip-Scott commented 8 years ago

Instead of having the icons be it's own theme, they should get installed in /usr/share/icons/elementary so the user doesn't need to change system themes :)

rhoconlinux commented 8 years ago

wel... that's not a good idea at all for several reasons. But first of all the reason is that this theme inherits the original elementary theme, allowing to receive updates (if there are some) on the original icons and adding those that are not present on it. But... since indeed IS another icon theme, it's ok to have it separately, right? :P

Philip-Scott commented 8 years ago

Since it's part of the elementary theme, it makes scene for it to be installed in the same directory, and anyone using the default elementary theme would just get the icons applied to them.

The elementary+ theme does not overwrite any system icons, so there wouldn't be a problem with either this or the official theme's update.

Also like i said, it makes it so everything get's integrated faster without having to use third party tools to select the theme, or have problems when if you uninstall the theme the whole system breaks

rhoconlinux commented 8 years ago

well... indeed, you wouldn't be able to uninstall it (without removing elementary theme)... et voilà :)

Proposing close.

Cheers!

quassy commented 8 years ago

I agree with @rhoconlinux. While we try not to overwrite any elementary icon, we can't be certain this is the case in the future, and mixing our work with the official elementary icon theme just feels like bad practice, there is a reason inheritage exists (elementary itself also inherits from hicolor). It would make manually removing elementary+ damn hard. Also at least on elementary you don't need any 3rd party tools, elementary+ Configurator (which is part of the PPA) supports changing the icon theme in one click.

Philip-Scott commented 8 years ago

Alright, sounds fair :) I'll close the issue then