b4j4 / indicator-kdeconnect

AppIndicator for KDE Connect
GNU Lesser General Public License v2.1
372 stars 37 forks source link

Monochrome icons on tray #1

Closed varlesh closed 8 years ago

varlesh commented 8 years ago

screenshot from 2016-11-13 15-26-36 Please change icons on tray. If you need help, i can draw new icons :) I'm not found icon names for tray on source files (indicator use this icons from dbus?)

varlesh commented 8 years ago

And thx for your work :+1: , i'm already create PPA for Ubuntu 16.04:

sudo add-apt-repository ppa:varlesh-l/indicator-kdeconnect
sudo apt update
sudo apt install kdeconnect indicator-kdeconnect
b4j4 commented 8 years ago

Great, i'm a fedora user, but i now a PPA will be useful for Ubuntu users. At this time it use default icons that is provide by dbus, and dbus get them from "/usr/share/icons/hicolor/scalable/status/" screenshot from 2016-11-13 20-53-57

But in the data folder have icons that was suppose to be the default icons, from the original project. Personally i think that icons is ugly, maybe you can redraw them and we can think in one way to use them by default.

varlesh commented 8 years ago

Ok, im add smartphone-connected from my theme - that working, icon apply from icon theme :) Use only smartphone-connected icon or this used too?: laptop-connected tablet-connected

varlesh commented 8 years ago

Now i'm draw 2 variants - ubuntu-mono-dark and and ubuntu-mono-light (because some users use white panels, some black)... and sizes 22px and 24px. Icon name will be as original. You like this variant?

b4j4 commented 8 years ago

Yes, if you have a laptop or a tablet with kdeconnect they will be apear on tray whit these icons. For users that use this variant of theme make sense have these 2 variants, like in my case i prefer light ones.

varlesh commented 8 years ago

Fine, i'm draw!

varlesh commented 8 years ago

image Also dark version too. Wait new PR soon :)

varlesh commented 8 years ago

@Bajoja I see on your screenshot use smartphone-disconnected icon or smartphone-trusted. This icons used too on tray? I can update and add new icons, if you need.

varlesh commented 8 years ago

Because i'm create only:

If you need -trusted and/or -disconected icons - i can added.

varlesh commented 8 years ago

Add all icons https://github.com/Bajoja/indicator-kdeconnect/pull/3 Merge please

b4j4 commented 8 years ago

Yes these icons are used too.

varlesh commented 8 years ago

Ok, now all complete. I'm close, need update only CMakeFiles and add depend ubuntu-mono

b4j4 commented 7 years ago

@varlesh Build the new version on PPA please, ;)

b4j4 commented 7 years ago

@varlesh Build the new version on PPA please, ;)