Re: [dev] [patch] map new window *after* arrange()

2011-07-09 Thread Peter Hartlich
> To prevent the first restack, maybe something like the attached patch? ... which causes the title bar to flicker, as restack calls drawbar before bailing out. Not sure why that's needed. Peter

Re: [dev] [patch] map new window *after* arrange()

2011-07-09 Thread Peter Hartlich
> just one annoying issue with floating apps. eg. running save as in gimp, > will open a window in the background. Oh man, sorry. I should've tested it more with floating windows. So arrange calls arrangemon, which calls restack, but the new floating window hasn't become selmon->sel yet. Restack

Re: [dev] [patch] map new window *after* arrange()

2011-07-09 Thread Bogdan IonuČ›
On Sat, Jul 9, 2011 at 10:01, garbeam wrote: > Hi Peter, > > On 9 July 2011 00:40, Peter Hartlich wrote: > >> I reviewed your patch and concluded I can't apply it as is, because > >> focus() would be called twice, once on the unmapped window, and then > >> on again. > > > > It seems to me that f

Re: [dev] [patch] map new window *after* arrange()

2011-07-09 Thread garbeam
Hi Peter, On 9 July 2011 00:40, Peter Hartlich wrote: >> I reviewed your patch and concluded I can't apply it as is, because >> focus() would be called twice, once on the unmapped window, and then >> on again. > > It seems to me that focusing and arranging should be independent > operations. So h