search

lxqt / libqtxdg

Qt implementation of freedesktop.org xdg specs
https://lxqt.github.io
GNU Lesser General Public License v2.1
72 stars 35 forks source link

Follow `freedesktop.org` in adding dirs under `DefaultDirectoryDirs` #285

Closed tsujan closed 2 years ago

tsujan commented 2 years ago

Quoting https://specifications.freedesktop.org/menu-spec/menu-spec-1.0.html: "The default locations that are earlier in the search path go later in the <Menu> so that they have priority".

tsujan commented 2 years ago

Yes, it seems priorities are OK now. Here, a panel restart isn't needed when adding/removing files — like in pcmanfm-qt → Applications.

stefonarch commented 2 years ago

Here, a panel restart isn't needed when adding/removing files

Yes, it's on the fly, the menu is generated at opening it seems to me. But I had to rebuild lxqt-panel against the branch.

tsujan commented 2 years ago

I asked for a review to make sure that this interpretation of freedesktop.org was correct (I assume the answer was positive). Feel free to make changes to the code style later. Merging...