nixos: Add a menu launcher for the NixOS manual #23861
Conversation
|
Nice idea. |
|
I'm not sure it's worth bothering; the desktopItem is a small inert text file that doesn't drag in any additional dependencies. |
|
Although, once we add a better icon it will pull that in, so maybe you're right. |
benley
force-pushed
the
nixos-manual-launcher
branch
from
cce2819
to
7494418
Compare
|
done :) |
| genericName = "View NixOS documentation in a web browser"; | ||
| # TODO: find a better icon (Nix logo + help overlay?) | ||
| icon = "system-help"; | ||
| exec = "xdg-open ${entry}"; |
There was a problem hiding this comment.
hm. Can I actually assume that xdg-open will be present if this is used?
There was a problem hiding this comment.
I guess it should be if at least one desktop env is installed system-wide.
On a server with no X the xdg-open command is not available. But since you just made the creation of the desktop item conditional it should be fine.
There was a problem hiding this comment.
Desktop entries specifically allow URLs, so I guess we might just use file://${entry} URL instead of xdg-open.
There was a problem hiding this comment.
Another way would be to execute ${help}, as we define it above anyway...
There was a problem hiding this comment.
re URLs, I am pretty sure that the URL field only applies to desktop files with Type=Link. And for some reason the gnome and KDE app menus refuse to work with those; they require Type=Application, which in turn requires an exec line. I'll switch to having it exec ${help} though, that's a good idea.
There was a problem hiding this comment.
Yes, I meant Type=Link, but I just read the specs now and never used that – thanks.
benley
force-pushed
the
nixos-manual-launcher
branch
from
7494418
to
638e1b8
Compare
|
updated, probably mergeable? |
This adds a clickable app launcher in most desktop environments to open the current system's generated documentation set.