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

Invalid desktop file (missing icon) #1402

Closed
stanislav-brabec opened this issue Jun 21, 2018 · 2 comments
Closed

Invalid desktop file (missing icon) #1402

stanislav-brabec opened this issue Jun 21, 2018 · 2 comments

Comments

@stanislav-brabec
Copy link
Contributor

Problem Description

src/tools/org.opensc.notify.desktop.in uses Icon=preferences-system-notifications without installing any icon. It is acceptable only for icons that are included in the Icon Naming Specification[1], section Application. preferences-system-notifications is not there. It exists only in GNOME specific themes. But org.opensc.notify.desktop.in does not contain OnlyShowIn=GNOME.

References:
[1] https://standards.freedesktop.org/icon-naming-spec/icon-naming-spec-latest.html

Proposed Resolution

Use an icon defined in the Icon Naming Specification, e. g. preferences-other
or
Install custom icon (or embed GNOME icon to the package).

Additionally, it is recommended to add GenericName, e. g.
`GenericName=Smart card notification

Steps to reproduce

Use other desktop environment than GNOME.
Application has no icon.

Logs

N/A

@frankmorgner
Copy link
Member

GenericName is not required nor do I see a recommendation for it anywhere...

@stanislav-brabec
Copy link
Contributor Author

GenericName is not required, but it is recommended. Many desktop systems display that GenericName together with the Name. It makes life easier to people who don't know names of applications. (Yes, Comment contains details as well, but it is typically displayed only as a pop-up.)

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

No branches or pull requests

2 participants