Re: [Patch] apps-menu

> Good catch,  the apps-menu doesn't currently honor the OnlyShowIn and 
> NotShowIn fields at the moment, this is something I plan on adding in 
> shortly.  This would require a check to see if, for example, KDE is 
> currently running or not, any suggestions as to that are welcome.
> I'll look into the options in respect to that to see if we can get 
> something working.  In the same vein, I'd like to be able to check 
> which, if any, DE is running when the apps-menu is generated in order to 
> give more appropriate menu entries, especially for KDE, but also for Gnome.

That's no big problem, sawfish does already check for the DE on startup, see
this code from defaults.jl:

(unless batch-mode

;; if it looks like GNOME is the desktop environment, then load the
;; extra GNOME integration module
      (require 'sawfish.wm.integration.gnome)

;; if it looks like KDE is the desktop environment, then load the
;; extra KDE integration module
  (if (getenv "KDE_SESSION_VERSION")
      (require 'sawfish.wm.integration.kde))))

So you should add some defvar to sawfish.wm.integration.[gnome|kde] ... though I'm not
yet sure which file is read first defaults.jl or apps{-,}menu.jl. Ahh, checks are for
GNOME 2x and KDE 4x. KDE 3x might have different envvar, but we don't have an integration
module for it.


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]