Skip to content
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

Hardcoded icon path in launcher #119

Closed
palob opened this issue Mar 28, 2015 · 2 comments
Closed

Hardcoded icon path in launcher #119

palob opened this issue Mar 28, 2015 · 2 comments

Comments

@palob
Copy link

palob commented Mar 28, 2015

Hi,

currently the icon path specified in the created launcher file is set to:
Icon=/usr/share/syncthing-gtk/icons/st-logo-128.png

(see https://github.com/syncthing/syncthing-gtk/blob/da715b52d71cb50359bdf3abc7e58154786b8b3d/syncthing-gtk.desktop)

This prevents icon theming without editing the launcher file.

I'd like to ask you to change the icon location as well as the icon path line in the launcher according to the freedesktop.org standards.

Please find more info here:
http://standards.freedesktop.org/icon-theme-spec/icon-theme-spec-latest.html
https://github.com/Foggalong/hardcode-fixer/wiki/What,-Why-&-How


According to the above resources standard icon locations are:

/usr/share/icons/hicolor/[size]/apps/[icon name]
~/.local/share/icons/hicolor/[size]/apps/[icon name]
/usr/share/pixmaps/[icon name]

The standard icon line in the .desktop launcher is:
Icon=[icon name] (without full path, without file extension)

for instance
Icon=syncthing-gtk
(for an icon syncthing-gtk.svg or syncthing-gtk.png stored in standard location)

Could you please move the icon to one of those and adjust the icon path in the .desktop file accordingly?

@kozec
Copy link
Owner

kozec commented Mar 28, 2015

Sounds reasonable. I'll try to not forget about that in next release.

@kozec kozec closed this as completed in c64400c Mar 28, 2015
@palob
Copy link
Author

palob commented Mar 28, 2015

Thanks a lot and keep up the good work!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants