Following my own ML message, the attached patch makes libxfce4util desktop entry parsing behaves as described: - If an key is set in the [Desktop Entry] section, then prefer/use the value from there - If an entry is set out of the [Desktop Entry] section, then use that value as a fallback. GNOME menu seems to do something similar.
Created attachment 592 The patch mentionned above.
Hm, that doesn't seem right. The menu should only care for [Desktop Entry], the [Desktop Action...]s are only for the file manager, IMHO. Isn't there a category for screensavers that can be ignored in xfdesktop?
Well, it works. From the desktop files, there is no "screensaver" category.
Created attachment 603 Example of a desktop file that shouldn't show Benny, I fail to undertand what could be the meaning of "NoDisplay=true" in a desktop action... I attach an example of an offending file as an example. So the change in the parsing in the patch provided will simply make options such as "NoDisplay=true" visible, that wouldn't override any option set in the "Desktop entry". I think we should fix that, one way or another. And I fail to see another way, but by supporting the "directory" option (and we currently don't).
Dunno... I'm still pretty sure that this should not be addressed at the .desktop parser level, but is the job of the menu implementation. Maybe for 4.4 xfdesktop could simply blacklist several well known .desktop files (like the KDE screensavers)? Xfce 4.6 will hopefully include a more standard compliant menu implementation that will be able to handle this.
Anyway it needs to be fixed.
Ok, I've implemented it in the module menu instead, I'll post a bug and a patch to Brian...