all repos — openbox @ d48e720c3977d1d64620108589380919b430affe

openbox fork - make it a bit more like ryudo

client_validate should return FALSE only for UnmapNotifies that will cause the window to become unmanaged
Dana Jansens danakj@orodu.net
commit

d48e720c3977d1d64620108589380919b430affe

parent

75ca8467921f68fdbd6100a1a435a410ef31acb0

1 files changed, 22 insertions(+), 2 deletions(-)

jump to
M openbox/client.copenbox/client.c

@@ -3591,17 +3591,37 @@ }

return NULL; } +static gboolean client_validate_unmap(ObClient *self, int n) +{ + XEvent e; + gboolean ret = TRUE; + + if (XCheckTypedWindowEvent(ob_display, self->window, UnmapNotify, &e)) { + if (n < self->ignore_unmaps) // ignore this one, but look for more + ret = client_validate_unmap(self, n+1); + else + ret = FALSE; // the window is going to become unmanaged + + /* put them back on the event stack so they end up in the same order */ + XPutBackEvent(ob_display, &e); + } + + return ret; +} + gboolean client_validate(ObClient *self) { XEvent e; XSync(ob_display, FALSE); /* get all events on the server */ - if (XCheckTypedWindowEvent(ob_display, self->window, DestroyNotify, &e) || - XCheckTypedWindowEvent(ob_display, self->window, UnmapNotify, &e)) { + if (XCheckTypedWindowEvent(ob_display, self->window, DestroyNotify, &e)) { XPutBackEvent(ob_display, &e); return FALSE; } + + if (!client_validate_unmap(self, 0)) + return FALSE; return TRUE; }