search

b4j4 / indicator-kdeconnect

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

Fixed accented character bug and changed the port to 8013 #61

Closed OzymandiasTheGreat closed 7 years ago

b4j4 commented 7 years ago

@OzymandiasTheGreat what you think about the sync button icon, aesthetically I do not think it looks beautiful, there is another on gtk sources that can be used like the reload button or do you think it should stay like is it?

OzymandiasTheGreat commented 7 years ago

@Bajoja I don't have strong opinions on this. I'm using numix icon theme, so current icon looks fine for me, but if it looks bad on other themes we can change it.

b4j4 commented 7 years ago

@OzymandiasTheGreat my bad, i figure it out that maybe a icon theme problem, is not necessary change that.

OzymandiasTheGreat commented 7 years ago

@Bajoja Nah, it's a minor change. Reload icon also looks fine, so if it looks better on more themes I think that's a good thing and we should change it.

b4j4 commented 7 years ago

Very ugly on my system, but i test it with numix and looks great. ugly

OzymandiasTheGreat commented 7 years ago

@Bajoja Yeah, that is not very pretty. I'll change it.

b4j4 commented 7 years ago

Ok