make some funcs const, and don't try focus a client if it is not visible
Dana Jansens danakj@orodu.net
1 files changed,
13 insertions(+),
4 deletions(-)
jump to
M
src/client.cc
→
src/client.cc
@@ -1082,9 +1082,12 @@ frame->adjustSize();
} -bool OBClient::focus() +bool OBClient::focus() const { - if (!(_can_focus || _focus_notify)) return false; + // won't try focus if the client doesn't want it, or if the window isn't + // visible on the screen + if (!(frame->isVisible() && (_can_focus || _focus_notify))) return false; + if (_focused) return true; if (_can_focus)@@ -1112,7 +1115,7 @@ return true;
} -void OBClient::unfocus() +void OBClient::unfocus() const { if (!_focused) return;@@ -1269,8 +1272,14 @@ the window manager never sees it because an unmap event is not sent
to an already unmapped window. */ + // we don't want the reparent event, put it back on the stack for the X + // server to deal with after we unmanage the window + XEvent ev; + ev.xreparent = e; + XPutBackEvent(otk::OBDisplay::display, &ev); + // this deletes us etc - Openbox::instance->screen(_screen)->unmanageWindow(this, true); + Openbox::instance->screen(_screen)->unmanageWindow(this); } }