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 for GNU/Linux systems #1111

Closed
Tracked by #221
palob opened this issue Mar 28, 2015 · 3 comments · Fixed by #1112
Closed
Tracked by #221

Hardcoded icon path in launcher for GNU/Linux systems #1111

palob opened this issue Mar 28, 2015 · 3 comments · Fixed by #1112

Comments

@palob
Copy link
Contributor

palob commented Mar 28, 2015

Hi,

currently the icon path specified in the created .desktop launcher file is set to:
Icon=/usr/share/app-install/icons/electrum.png

(see https://github.com/spesmilo/electrum/blob/bc3013caf0d3d6a35290e9cc9e51125b7d03d14c/electrum.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=electrum
(for an icon electrum.svg or electrum.png stored in standard location)

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

@ecdsa
Copy link
Member

ecdsa commented Mar 28, 2015

sure, can you write a pull request?

@palob
Copy link
Contributor Author

palob commented Mar 28, 2015

Alright, will do it tomorrow, touching setup.py and electrum.desktop.

@ecdsa
Copy link
Member

ecdsa commented Jan 26, 2016

@palob I sent you a mail, did you receive it?

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

Successfully merging a pull request may close this issue.

2 participants