On Tue, May 28, 2002 at 12:57:35PM +0200, Wilbert Berendsen wrote:
> 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.hmm.. except that you in fact, shouldnt be able to send the window to the current workspace :) Whereas, you can choose other windows from the window list of the current workspace. xOr -- I am damn unsatisfied to be killed in this way.
msg07065/pgp00000.pgp
Description: PGP signature
