Re: [dwm] max_and_focus.patch regression

On Mon, Sep 04, 2006 at 05:34:33PM +0200, Sander van Dijk wrote:
> On 9/4/06, Anselm R. Garbe <arg_AT_10kloc.org> wrote:
> >On Mon, Sep 04, 2006 at 05:18:17PM +0200, Sander van Dijk wrote:
> >> if(isvisible(c)) {
> >> focus(NULL);
> >> sel = c;
> >> }
> >>
> >> for instance, or by manually grabbing the right buttons for the old
> >> focused client. Not sure what the right way is (calling focus on c
> >> itself at that point is probably not an option, since that would
> >> incorrectly map its twin afterwards).
> >
> >Your last remark seems best for now.
> >
> >Included in dwm-1.3.
>
> Maybe it's even better to first ban the new client, than map it's
> windows, than focus it and then call arrange(). My guess is that would
> be cleaner
>
> ban(c);
> if(isvisible(c))
> focus(c);
> mapstuff;
> arrange();

But with this proposal the map stuff should be before the
isvisible(c) check. Otherwise focus won't have much effect...

Regards,

-- 
 Anselm R. Garbe  ><><  www.ebrag.de  ><><  GPG key: 0D73F361