-
-
Notifications
You must be signed in to change notification settings - Fork 764
Add a XDG desktop file #354
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
Conversation
This allows proper packaging of the application for Linux distros
benjamin051000
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I ran desktop-file-validate, which I found from the Arch Wiki:
❯ desktop-file-validate arnis.desktop
arnis.desktop: hint: value "Game;Utility;Geoscience" for key "Categories" in group "Desktop Entry" contains more than one main category; application might appear more than once in the application menu
arnis.desktop: hint: value item "Geoscience" in key "Categories" in group "Desktop Entry" can be extended with another category among the following categories: Education, or Science
Please fix the icon and see other comments (may or may not require changes)
| Name=Arnis | ||
| Comment=Generate cities from real life in Minecraft | ||
| Comment[lt]=Sugeneruokite tikrovės miestus „Minecraft“ žaidime | ||
| Icon=arnis |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think arnis is an Icon. But there are icons in https://github.com/louis-e/arnis/blob/main/icons/, maybe you could specify one of those?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, when running desktop-file-install arnis.desktop, I don't see the icon.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The .desktop file is intended as an alternative to the tauri bundle workflow (so the packager/advanced user can definitely put the icon in the correct place)
| Icon=arnis | ||
| Exec=arnis | ||
| Categories=Game;Utility;Geoscience | ||
| Keywords=OSM;OpenStreetMap;Tauri |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What do these keywords do? I see this in the XDG Desktop Entry Specification:
A list of strings which may be used in addition to other metadata to describe this entry. This can be useful e.g. to facilitate searching through entries. The values are not meant for display, and should not be redundant with the values of Name or GenericName.
Does this provide additional keywords to discover the app when using the OS's search tool (e.g., KRunner or GNOME search)?
If I understand what this does correctly, I don't think arnis should show up if someone searches "Tauri" in their OS search bar, right? It's akin to adding "rust" to this field. I guess "OSM"/"OpenStreetMap" are okay. What about "Minecraft"?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does this provide additional keywords to discover the app when using the OS's search tool
I think that's the main reason for that option
|
Since there hasn't been any more commits to resolve the discussion, I will merge this and follow up with an own commit to fix the open points |
This allows proper packaging of the application for Linux distros