diff options
| author | Dana Jansens <danakj@orodu.net> | 2011-05-13 15:48:53 -0400 |
|---|---|---|
| committer | Dana Jansens <danakj@orodu.net> | 2011-05-13 15:52:17 -0400 |
| commit | cf276fafd853356a71fb99e0c60a08c034b0d537 (patch) | |
| tree | 4830d6d794474a152b440cae7a415143666a4ecb /openbox | |
| parent | 823997f0011b0c948ebd9d3a3b155884a336ed2f (diff) | |
Fix Java apps always appearing on desktop 0.
Fake managing a window doesn't read a requested desktop, but ended up placing
a NET_WM_DESKTOP hint on the window (with value 0). Fake managing doesn't
need to set the DESKTOP hint since the window is not actually being managed,
so remove it from the codepath.
Diffstat (limited to 'openbox')
| -rw-r--r-- | openbox/client.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/openbox/client.c b/openbox/client.c index 6a169336..1b010e4b 100644 --- a/openbox/client.c +++ b/openbox/client.c @@ -437,6 +437,10 @@ void client_manage(Window window, ObPrompt *prompt) client_apply_startup_state(self, place.x, place.y, place.width, place.height); + /* set the initial value of the desktop hint, when one wasn't requested + on map. */ + OBT_PROP_SET32(self->window, NET_WM_DESKTOP, CARDINAL, self->desktop); + /* grab mouse bindings before showing the window */ mouse_grab_for_client(self, TRUE); @@ -2792,9 +2796,6 @@ static void client_apply_startup_state(ObClient *self, self->area = oldarea; client_configure(self, x, y, w, h, FALSE, TRUE, FALSE); - /* set the desktop hint, to make sure that it always exists */ - OBT_PROP_SET32(self->window, NET_WM_DESKTOP, CARDINAL, self->desktop); - /* nothing to do for the other states: skip_taskbar skip_pager |