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

Add a XDG desktop file #354

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

TheComputerGuy96
Copy link
Contributor

This allows proper packaging of the application for Linux distros

This allows proper packaging of the application for Linux distros
Copy link
Contributor

@benjamin051000 benjamin051000 left a 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
Copy link
Contributor

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?

Copy link
Contributor

@benjamin051000 benjamin051000 Feb 1, 2025

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.

Copy link
Contributor Author

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
Copy link
Contributor

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"?

Copy link
Contributor Author

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

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 this pull request may close these issues.

2 participants