Hoi xOr,
Op Tue, 28 May 2002 05:51:57 -0500
schreef xOr <[EMAIL PROTECTED]>:
> On Tue, May 28, 2002 at 12:48:57PM +0200, Wilbert Berendsen wrote:
> > Hi all,
> >
> > Here's a little patch to highlight current workspace in Send To menu.
> > Comes in handy when you want to send a window to the next right
> > workspace but don't know which workspace you are now (due to hidden
> > toolbar).
> >
> > --- /home/wilbert/src/cvs/blackbox/src/Windowmenu.cc Wed May 15
> > 23:38:56 2002
> > +++ src/Windowmenu.cc Tue May 28 12:44:41 2002
> > @@ -190,6 +190,7 @@
> > } else {
> > changeItemLabel(i, getScreen()->getWorkspace(i)->getName());
> > }
> > + setItemSelected(i, i == getScreen()->getCurrentWorkspaceID());
> > }
> >
> > Basemenu::update();
>
> Would it not be better to setItemEnabled() to false on the item (disable
> it)?
I thought of that first, but considered greying out the item a bit
unfriendly ("you CAN'T send the window to the CUrrent WORKSpace, you
Stupid":).
Making it look selected is the same as in the Workspace menu. That one
also highlights the current workspace.
Groetjes,
Wilbert
--
Wilbert Berendsen (http://www.xs4all.nl/~wbsoft/)
http://www.stoppoliceware.org/ http://digitalspeech.org/
