Browse Source

handle OnlyShowIn and NotShowIn better

While the target WMs are not registered with freedesktop, some
    users do use these fields for these WMs.
    https://github.com/search?l=&q=onlyshowin%3D.%2Afluxbox+extension%3Adesktop&type=Code
    returns 367 code results.
pull/19/head
B Stack 4 years ago
parent
commit
62fb13e154
  1. 8
      src/xdgmenumaker

8
src/xdgmenumaker

@ -400,11 +400,11 @@ def get_entry_info(desktopfile, ico_paths=True):
hidden = de.getHidden() hidden = de.getHidden()
nodisplay = de.getNoDisplay() nodisplay = de.getNoDisplay()
# none of the freedesktop registered environments are supported by # none of the freedesktop registered environments are supported by
# OnlyShowIn anyway: # OnlyShowIn but it might be worth using some extra logic here.
# http://standards.freedesktop.org/menu-spec/latest/apb.html # http://standards.freedesktop.org/menu-spec/latest/apb.html
# So if OnlyShowIn is set, it certainly isn't for any of the WMs if (onlyshowin != [] and not (desktop.lower() in (name.lower() for name in onlyshowin))) \
# xdgmenumaker supports. or (desktop.lower() in (name.lower for name in notshowin)) \
if (onlyshowin != []) or (desktop in notshowin) or hidden or nodisplay: or hidden or nodisplay:
return None return None
name = de.getName().encode('utf-8') name = de.getName().encode('utf-8')

Loading…
Cancel
Save