On Fri, 2006-08-11 at 19:41 +0200, Sander van Dijk wrote:
> On 8/11/06, Ross Mohn <rpmohn_AT_waxandwane.org> wrote:
> > OK, I love this feature! How about making Button3 a toggle:
> >
> > diff -u event.000.c event.c
> > --- event.000.c 2006-08-11 13:03:10.000000000 -0400
> > +++ event.c 2006-08-11 13:17:07.000000000 -0400
> > @@ -109,7 +109,7 @@
> > x += textw(tags[a.i]);
> > if(ev->x < x) {
> > if(ev->button == Button3) {
> > - tsel[a.i] = True;
> > + tsel[a.i] = tsel[a.i] ? False : True;
> > arrange(NULL);
> > drawall();
> > }
>
> Maybe, I'll have to try it a bit longer to see if I like it. I think
> it probably needs an extra check though, since I doubt it's a good
> idea to allow deselecting all tags (that could probably lead to nasty
> cornercases; then again, maybe it makes sense to be able to display
> nothing at all). I think btw. that with this multi tag viewing,
> viewnext() and viewprev() become obsolete (there's no such thing as a
> next view when you're viewing two tags).
I tried the toggle and I like it. I wasn't sure how it would be to
deselect everything, but it actually feels ok to me. It's like cleaning
off your entire physical desktop. Doesn't seem to cause any programmatic
issues either.
-RPM
Received on Fri Aug 11 2006 - 20:21:28 UTC
This archive was generated by hypermail 2.2.0 : Sun Jul 13 2008 - 14:30:06 UTC