all repos — openbox @ fd5369cd1783705e0422b3b9b047a4d041ac264b

openbox fork - make it a bit more like ryudo

only send configu notify if the window is mapped (has a frame)
Dana Jansens danakj@orodu.net
commit

fd5369cd1783705e0422b3b9b047a4d041ac264b

parent

08dc1566395412dc9f6ce85e063455e654f87af6

1 files changed, 18 insertions(+), 16 deletions(-)

jump to
M src/client.ccsrc/client.cc

@@ -1003,24 +1003,26 @@ {

_area.setPos(x, y); // move the frame to be in the requested position - if (frame) // this can be called while mapping, before frame exists + if (frame) { // this can be called while mapping, before frame exists frame->adjustPosition(); - // send synthetic configure notify - XEvent event; - event.type = ConfigureNotify; - event.xconfigure.display = otk::OBDisplay::display; - event.xconfigure.event = _window; - event.xconfigure.window = _window; - event.xconfigure.x = x; - event.xconfigure.y = y; - event.xconfigure.width = _area.width(); - event.xconfigure.height = _area.height(); - event.xconfigure.border_width = _border_width; - event.xconfigure.above = frame->window(); - event.xconfigure.override_redirect = False; - XSendEvent(event.xconfigure.display, event.xconfigure.window, False, - StructureNotifyMask, &event); + // send synthetic configure notify (we don't need to if we aren't mapped + // yet) + XEvent event; + event.type = ConfigureNotify; + event.xconfigure.display = otk::OBDisplay::display; + event.xconfigure.event = _window; + event.xconfigure.window = _window; + event.xconfigure.x = x; + event.xconfigure.y = y; + event.xconfigure.width = _area.width(); + event.xconfigure.height = _area.height(); + event.xconfigure.border_width = _border_width; + event.xconfigure.above = frame->window(); + event.xconfigure.override_redirect = False; + XSendEvent(event.xconfigure.display, event.xconfigure.window, False, + StructureNotifyMask, &event); + } }