On Mon, Jul 9, 2012 at 2:25 PM, Raghavendra D Prabhu < raghu.prabhu13@gmail.com> wrote:
So, after the previous post I did some digging in the code. getTransientForHint is from Graphics.X11.Xlib.Extras haskell library.
Yes. Do not expect that the X11 binding will be changed to add this; that's the wrong place to put it.
So there are two ways to fix this: 1. Fix in libX11 by adding another function or fixing existing one. 2. Fixing getTransientForHint in G.X.X11.Extras by adding another condition to check for WM_CLIENT_LEADER according to ICCWM specs.
And then there's the correct one: extend the *user* of this function, in ManageHelpers, to also do the WM_CLIENT_LEADER check. Modifying a low level X11 binding to do something the bound library does not, namely adding WM_CLIENT_LEADER to a function which is documented to only do WM_TRANSIENT_FOR, is the wrong place to put this. -- brandon s allbery allbery.b@gmail.com wandering unix systems administrator (available) (412) 475-9364 vm/sms