I-Dream-in-Code / kde-arch-update-plasmoid

30 stars 9 forks source link

The icon doesn't match the Breeze theme #20

Closed NerosTie closed 6 years ago

NerosTie commented 6 years ago

I think the default icon is too "Arch" and it should more like this one for example:

screenshot_20180224_104452

ghost commented 6 years ago

Well, Arch is definitely the target distro here, it's even included in project name. The above icon is already used by Discover so reusing it isn't advisable. If other Arch-like distros want to package this, they can use icon of their choice.

NerosTie commented 6 years ago

Yes but the actual icon doesn't look like an update icon and it's doesn't match well with the Breeze theme. I don't tell to use the Discover's icon, but something more like this one for this plasmoid.

ghost commented 6 years ago

This is probably very subjective as it looks good in Breeze for me but I don't oppose adding new icons.

I-Dream-in-Code commented 6 years ago

It's an arch icon because only arch requires check updates and manual upgrade.

Gnome arch update notifier uses a similar arch logo icon

I-Dream-in-Code commented 6 years ago

@NerosTie custom icon support has been added