joewing / jwm

Joe's Window Manager
http://joewing.net/projects/jwm
MIT License
517 stars 85 forks source link

_NET_RESTACK_WINDOW not supported #118

Closed mse2 closed 10 years ago

mse2 commented 10 years ago

It has been reported that MSEgui applications: http://sourceforge.net/projects/mseide-msegui/ don't work well with jwm 2.2.2. It seems that _NET_RESTACK_WINDOW is not supported. The test project is here: https://gitorious.org/mseuniverse/mseuniverse/source/testcase/window/netrestackwindow

bbidulock commented 10 years ago

A couple of problems with your test program. Per wm-spec-1.5, _NET_RESTACK_WINDOW is only sent by pagers. Are you a pager? Also, you are setting the source id to 0. It must be set to 2 when it comes from a pager. Also, _NET_RESTACK_WINDOW support is not required and jwm does not put it in _NET_SUPPORTED. jwm has its own pager.

Window managers that do not support _NET_RESTACK_WINDOW include: mutter, uwm, awesome, waimea, wind, jwm, pekwm, blackbox, windowmaker, afterstep, ... The only ones I know of that do are: adwm, matwm2, fluxbox, icewm, openbox, metacity.

joewing commented 10 years ago

I agree that only pager-type programs should use _NET_RESTACK_WINDOW, but I think it would be good to allow JWM to support this at some point to allow other pager-type programs to work.

bbidulock commented 10 years ago

I don't know of any pagers that actually use it. libwnck doesn't.

joewing commented 10 years ago

This should work now. I'm not completely sure it works correctly since I don't know of an existing program to use it.

mse2 commented 10 years ago

Tested with MSEide --RESTACKWINDOW --NORECONFIGUREWMWINDOW, looks OK, thanks a lot! https://gitorious.org/mseide-msegui http://sourceforge.net/projects/mseide-msegui/