Skip to content

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

@louis-e
Copy link
Owner

louis-e commented Apr 26, 2025

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

@louis-e louis-e merged commit f8a1da1 into louis-e:main Apr 26, 2025
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.

3 participants