X-Git-Url: https://git.dogcows.com/gitweb?a=blobdiff_plain;f=src%2Fclient.cc;h=eeb09a374981a4fe5cfe9dc30d3f6386c6faa980;hb=5face4c6f35172761367f63ac0b6eaf62d84e532;hp=95d1509562d114b6a7861d10807a7973ac10e31c;hpb=b02c91caad1ef6ddaebe8d58a410a089d47f1f59;p=chaz%2Fopenbox diff --git a/src/client.cc b/src/client.cc index 95d15095..eeb09a37 100644 --- a/src/client.cc +++ b/src/client.cc @@ -8,12 +8,14 @@ #include "frame.hh" #include "screen.hh" #include "openbox.hh" +#include "bindings.hh" #include "otk/display.hh" #include "otk/property.hh" extern "C" { #include #include +#include #include @@ -23,9 +25,9 @@ extern "C" { namespace ob { -OBClient::OBClient(int screen, Window window) - : otk::OtkEventHandler(), - OBWidget(OBWidget::Type_Client), +Client::Client(int screen, Window window) + : otk::EventHandler(), + WidgetBase(WidgetBase::Type_Client), frame(0), _screen(screen), _window(window) { assert(screen >= 0); @@ -35,132 +37,139 @@ OBClient::OBClient(int screen, Window window) // update EVERYTHING the first time!! - // the state is kinda assumed to be normal. is this right? XXX - _wmstate = NormalState; _iconic = false; - // no default decors or functions, each has to be enabled - _decorations = _functions = 0; + // we default to NormalState, visible + _wmstate = NormalState; // start unfocused _focused = false; + // not a transient by default of course + _transient_for = 0; + // pick a layer to start from + _layer = Layer_Normal; + // default to not urgent + _urgent = false; getArea(); getDesktop(); - // XXX: updateTransientFor(); - getType(); - - // set the decorations and functions - switch (_type) { - case Type_Normal: - // normal windows retain all of the possible decorations and - // functionality - _decorations = Decor_Titlebar | Decor_Handle | Decor_Border | - Decor_Iconify | Decor_Maximize; - _functions = Func_Resize | Func_Move | Func_Iconify | Func_Maximize; - case Type_Dialog: - // dialogs cannot be maximized - _decorations &= ~Decor_Maximize; - _functions &= ~Func_Maximize; - break; - - case Type_Menu: - case Type_Toolbar: - case Type_Utility: - // these windows get less functionality - _decorations &= ~(Decor_Iconify | Decor_Handle); - _functions &= ~(Func_Iconify | Func_Resize); - break; + updateTransientFor(); + getType(); + getMwmHints(); - case Type_Desktop: - case Type_Dock: - case Type_Splash: - // none of these windows are manipulated by the window manager - _decorations = 0; - _functions = 0; - break; - } - - getMwmHints(); // this fucks (in good ways) with the decors and functions getState(); getShaped(); updateProtocols(); - updateNormalHints(); - updateWMHints(); + + // got the type, the mwmhints, and the protocols, so we're ready to set up + // the decorations/functions + setupDecorAndFunctions(); + + getGravity(); // get the attribute gravity + updateNormalHints(); // this may override the attribute gravity + // also get the initial_state and set _iconic if we aren't "starting" + // when we're "starting" that means we should use whatever state was already + // on the window over the initial map state, because it was already mapped + updateWMHints(openbox->state() != Openbox::State_Starting); updateTitle(); updateIconTitle(); updateClass(); + updateStrut(); - calcLayer(); + // this makes sure that these windows appear on all desktops + if (/*_type == Type_Dock ||*/ _type == Type_Desktop) + _desktop = 0xffffffff; + + // set the desktop hint, to make sure that it always exists, and to reflect + // any changes we've made here + otk::Property::set(_window, otk::Property::atoms.net_wm_desktop, + otk::Property::atoms.cardinal, (unsigned)_desktop); + changeState(); } -OBClient::~OBClient() +Client::~Client() { - const otk::OBProperty *property = Openbox::instance->property(); - - if (Openbox::instance->state() != Openbox::State_Exiting) { + // clean up childrens' references + while (!_transients.empty()) { + _transients.front()->_transient_for = 0; + _transients.pop_front(); + } + + // clean up parents reference to this + if (_transient_for) + _transient_for->_transients.remove(this); // remove from old parent + + if (openbox->state() != Openbox::State_Exiting) { // these values should not be persisted across a window unmapping/mapping - property->erase(_window, otk::OBProperty::net_wm_desktop); - property->erase(_window, otk::OBProperty::net_wm_state); + otk::Property::erase(_window, otk::Property::atoms.net_wm_desktop); + otk::Property::erase(_window, otk::Property::atoms.net_wm_state); + } else { + // if we're left in an iconic state, the client wont be mapped. this is + // bad, since we will no longer be managing the window on restart + if (_iconic) + XMapWindow(**otk::display, _window); } } -void OBClient::getDesktop() +void Client::getGravity() { - const otk::OBProperty *property = Openbox::instance->property(); + XWindowAttributes wattrib; + Status ret; + + ret = XGetWindowAttributes(**otk::display, _window, &wattrib); + assert(ret != BadWindow); + _gravity = wattrib.win_gravity; +} + +void Client::getDesktop() +{ // defaults to the current desktop - _desktop = 0; // XXX: change this to the current desktop! + _desktop = openbox->screen(_screen)->desktop(); - property->get(_window, otk::OBProperty::net_wm_desktop, - otk::OBProperty::Atom_Cardinal, - &_desktop); + if (otk::Property::get(_window, otk::Property::atoms.net_wm_desktop, + otk::Property::atoms.cardinal, + (long unsigned*)&_desktop)) { +#ifdef DEBUG +// printf("Window requested desktop: %ld\n", _desktop); +#endif + } } -void OBClient::getType() +void Client::getType() { - const otk::OBProperty *property = Openbox::instance->property(); - _type = (WindowType) -1; unsigned long *val; unsigned long num = (unsigned) -1; - if (property->get(_window, otk::OBProperty::net_wm_window_type, - otk::OBProperty::Atom_Atom, - &num, &val)) { + if (otk::Property::get(_window, otk::Property::atoms.net_wm_window_type, + otk::Property::atoms.atom, &num, &val)) { // use the first value that we know about in the array for (unsigned long i = 0; i < num; ++i) { - if (val[i] == - property->atom(otk::OBProperty::net_wm_window_type_desktop)) + if (val[i] == otk::Property::atoms.net_wm_window_type_desktop) _type = Type_Desktop; - else if (val[i] == - property->atom(otk::OBProperty::net_wm_window_type_dock)) + else if (val[i] == otk::Property::atoms.net_wm_window_type_dock) _type = Type_Dock; - else if (val[i] == - property->atom(otk::OBProperty::net_wm_window_type_toolbar)) + else if (val[i] == otk::Property::atoms.net_wm_window_type_toolbar) _type = Type_Toolbar; - else if (val[i] == - property->atom(otk::OBProperty::net_wm_window_type_menu)) + else if (val[i] == otk::Property::atoms.net_wm_window_type_menu) _type = Type_Menu; - else if (val[i] == - property->atom(otk::OBProperty::net_wm_window_type_utility)) + else if (val[i] == otk::Property::atoms.net_wm_window_type_utility) _type = Type_Utility; - else if (val[i] == - property->atom(otk::OBProperty::net_wm_window_type_splash)) + else if (val[i] == otk::Property::atoms.net_wm_window_type_splash) _type = Type_Splash; - else if (val[i] == - property->atom(otk::OBProperty::net_wm_window_type_dialog)) + else if (val[i] == otk::Property::atoms.net_wm_window_type_dialog) _type = Type_Dialog; - else if (val[i] == - property->atom(otk::OBProperty::net_wm_window_type_normal)) + else if (val[i] == otk::Property::atoms.net_wm_window_type_normal) _type = Type_Normal; -// else if (val[i] == -// property->atom(otk::OBProperty::kde_net_wm_window_type_override)) -// mwm_decorations = 0; // prevent this window from getting any decor - // XXX: make this work again +// XXX: make this work again +// else if (val[i] == otk::Property::atoms.kde_net_wm_window_type_override) +// mwm_decorations = 0; // prevent this window from getting any decor + if (_type != (WindowType) -1) + break; // grab the first known type } delete val; } @@ -170,76 +179,124 @@ void OBClient::getType() * the window type hint was not set, which means we either classify ourself * as a normal window or a dialog, depending on if we are a transient. */ - // XXX: make this code work! - //if (isTransient()) - // _type = Type_Dialog; - //else + if (_transient_for) + _type = Type_Dialog; + else _type = Type_Normal; } } -void OBClient::getMwmHints() +void Client::setupDecorAndFunctions() { - const otk::OBProperty *property = Openbox::instance->property(); + // start with everything (cept fullscreen) + _decorations = Decor_Titlebar | Decor_Handle | Decor_Border | + Decor_AllDesktops | Decor_Iconify | Decor_Maximize; + _functions = Func_Resize | Func_Move | Func_Iconify | Func_Maximize | + Func_Shade; + if (_delete_window) { + _decorations |= Decor_Close; + _functions |= Func_Close; + } + + switch (_type) { + case Type_Normal: + // normal windows retain all of the possible decorations and + // functionality, and are the only windows that you can fullscreen + _functions |= Func_Fullscreen; - unsigned long num; - MwmHints *hints; + case Type_Dialog: + // dialogs cannot be maximized + _decorations &= ~Decor_Maximize; + _functions &= ~Func_Maximize; + break; - num = MwmHints::elements; - if (!property->get(_window, otk::OBProperty::motif_wm_hints, - otk::OBProperty::motif_wm_hints, &num, - (unsigned long **)&hints)) - return; - - if (num < MwmHints::elements) { - delete [] hints; - return; + case Type_Menu: + case Type_Toolbar: + case Type_Utility: + // these windows get less functionality + _decorations &= ~(Decor_Iconify | Decor_Handle); + _functions &= ~(Func_Iconify | Func_Resize); + break; + + case Type_Desktop: + case Type_Dock: + case Type_Splash: + // none of these windows are manipulated by the window manager + _decorations = 0; + _functions = 0; + break; } - // retrieved the hints // Mwm Hints are applied subtractively to what has already been chosen for // decor and functionality - - if (hints->flags & MwmFlag_Decorations) { - if (! (hints->decorations & MwmDecor_All)) { - if (! (hints->decorations & MwmDecor_Border)) + if (_mwmhints.flags & MwmFlag_Decorations) { + if (! (_mwmhints.decorations & MwmDecor_All)) { + if (! (_mwmhints.decorations & MwmDecor_Border)) _decorations &= ~Decor_Border; - if (! (hints->decorations & MwmDecor_Handle)) + if (! (_mwmhints.decorations & MwmDecor_Handle)) _decorations &= ~Decor_Handle; - if (! (hints->decorations & MwmDecor_Title)) + if (! (_mwmhints.decorations & MwmDecor_Title)) { _decorations &= ~Decor_Titlebar; - if (! (hints->decorations & MwmDecor_Iconify)) + // if we don't have a titlebar, then we cannot shade! + _functions &= ~Func_Shade; + } + if (! (_mwmhints.decorations & MwmDecor_Iconify)) _decorations &= ~Decor_Iconify; - if (! (hints->decorations & MwmDecor_Maximize)) + if (! (_mwmhints.decorations & MwmDecor_Maximize)) _decorations &= ~Decor_Maximize; } } - if (hints->flags & MwmFlag_Functions) { - if (! (hints->functions & MwmFunc_All)) { - if (! (hints->functions & MwmFunc_Resize)) + if (_mwmhints.flags & MwmFlag_Functions) { + if (! (_mwmhints.functions & MwmFunc_All)) { + if (! (_mwmhints.functions & MwmFunc_Resize)) _functions &= ~Func_Resize; - if (! (hints->functions & MwmFunc_Move)) + if (! (_mwmhints.functions & MwmFunc_Move)) _functions &= ~Func_Move; - if (! (hints->functions & MwmFunc_Iconify)) + if (! (_mwmhints.functions & MwmFunc_Iconify)) _functions &= ~Func_Iconify; - if (! (hints->functions & MwmFunc_Maximize)) + if (! (_mwmhints.functions & MwmFunc_Maximize)) _functions &= ~Func_Maximize; - //if (! (hints->functions & MwmFunc_Close)) + // dont let mwm hints kill the close button + //if (! (_mwmhints.functions & MwmFunc_Close)) // _functions &= ~Func_Close; } } + + changeAllowedActions(); +} + + +void Client::getMwmHints() +{ + unsigned long num = MwmHints::elements; + unsigned long *hints; + + _mwmhints.flags = 0; // default to none + + if (!otk::Property::get(_window, otk::Property::atoms.motif_wm_hints, + otk::Property::atoms.motif_wm_hints, &num, + (unsigned long **)&hints)) + return; + + if (num >= MwmHints::elements) { + // retrieved the hints + _mwmhints.flags = hints[0]; + _mwmhints.functions = hints[1]; + _mwmhints.decorations = hints[2]; + } + delete [] hints; } -void OBClient::getArea() +void Client::getArea() { XWindowAttributes wattrib; Status ret; - ret = XGetWindowAttributes(otk::OBDisplay::display, _window, &wattrib); + ret = XGetWindowAttributes(**otk::display, _window, &wattrib); assert(ret != BadWindow); _area.setRect(wattrib.x, wattrib.y, wattrib.width, wattrib.height); @@ -247,44 +304,36 @@ void OBClient::getArea() } -void OBClient::getState() +void Client::getState() { - const otk::OBProperty *property = Openbox::instance->property(); - _modal = _shaded = _max_horz = _max_vert = _fullscreen = _above = _below = - _skip_taskbar = _skip_pager = false; + _iconic = _skip_taskbar = _skip_pager = false; unsigned long *state; unsigned long num = (unsigned) -1; - if (property->get(_window, otk::OBProperty::net_wm_state, - otk::OBProperty::Atom_Atom, &num, &state)) { + if (otk::Property::get(_window, otk::Property::atoms.net_wm_state, + otk::Property::atoms.atom, &num, &state)) { for (unsigned long i = 0; i < num; ++i) { - if (state[i] == property->atom(otk::OBProperty::net_wm_state_modal)) + if (state[i] == otk::Property::atoms.net_wm_state_modal) _modal = true; - else if (state[i] == - property->atom(otk::OBProperty::net_wm_state_shaded)) + else if (state[i] == otk::Property::atoms.net_wm_state_shaded) _shaded = true; - else if (state[i] == - property->atom(otk::OBProperty::net_wm_state_skip_taskbar)) + else if (state[i] == otk::Property::atoms.net_wm_state_hidden) + _iconic = true; + else if (state[i] == otk::Property::atoms.net_wm_state_skip_taskbar) _skip_taskbar = true; - else if (state[i] == - property->atom(otk::OBProperty::net_wm_state_skip_pager)) + else if (state[i] == otk::Property::atoms.net_wm_state_skip_pager) _skip_pager = true; - else if (state[i] == - property->atom(otk::OBProperty::net_wm_state_fullscreen)) + else if (state[i] == otk::Property::atoms.net_wm_state_fullscreen) _fullscreen = true; - else if (state[i] == - property->atom(otk::OBProperty::net_wm_state_maximized_vert)) + else if (state[i] == otk::Property::atoms.net_wm_state_maximized_vert) _max_vert = true; - else if (state[i] == - property->atom(otk::OBProperty::net_wm_state_maximized_horz)) + else if (state[i] == otk::Property::atoms.net_wm_state_maximized_horz) _max_horz = true; - else if (state[i] == - property->atom(otk::OBProperty::net_wm_state_above)) + else if (state[i] == otk::Property::atoms.net_wm_state_above) _above = true; - else if (state[i] == - property->atom(otk::OBProperty::net_wm_state_below)) + else if (state[i] == otk::Property::atoms.net_wm_state_below) _below = true; } @@ -293,18 +342,18 @@ void OBClient::getState() } -void OBClient::getShaped() +void Client::getShaped() { _shaped = false; #ifdef SHAPE - if (otk::OBDisplay::shape()) { + if (otk::display->shape()) { int foo; unsigned int ufoo; int s; - XShapeSelectInput(otk::OBDisplay::display, _window, ShapeNotifyMask); + XShapeSelectInput(**otk::display, _window, ShapeNotifyMask); - XShapeQueryExtents(otk::OBDisplay::display, _window, &s, &foo, + XShapeQueryExtents(**otk::display, _window, &s, &foo, &foo, &ufoo, &ufoo, &foo, &foo, &foo, &ufoo, &ufoo); _shaped = (s != 0); } @@ -312,36 +361,47 @@ void OBClient::getShaped() } -void OBClient::calcLayer() { - if (_iconic) _layer = OBScreen::Layer_Icon; - else if (_type == Type_Desktop) _layer = OBScreen::Layer_Desktop; - else if (_type == Type_Dock) _layer = OBScreen::Layer_Top; - else if (_fullscreen) _layer = OBScreen::Layer_Fullscreen; - else if (_above) _layer = OBScreen::Layer_Above; - else if (_below) _layer = OBScreen::Layer_Below; - else _layer = OBScreen::Layer_Normal; +void Client::calcLayer() { + StackLayer l; + + if (_iconic) l = Layer_Icon; + else if (_fullscreen) l = Layer_Fullscreen; + else if (_type == Type_Desktop) l = Layer_Desktop; + else if (_type == Type_Dock) { + if (!_below) l = Layer_Top; + else l = Layer_Normal; + } + else if (_above) l = Layer_Above; + else if (_below) l = Layer_Below; + else l = Layer_Normal; + + if (l != _layer) { + _layer = l; + if (frame) { + /* + if we don't have a frame, then we aren't mapped yet (and this would + SIGSEGV :) + */ + openbox->screen(_screen)->raiseWindow(this); + } + } } -void OBClient::updateProtocols() +void Client::updateProtocols() { - const otk::OBProperty *property = Openbox::instance->property(); - Atom *proto; int num_return = 0; _focus_notify = false; - _decorations &= ~Decor_Close; - _functions &= ~Func_Close; + _delete_window = false; - if (XGetWMProtocols(otk::OBDisplay::display, _window, &proto, &num_return)) { + if (XGetWMProtocols(**otk::display, _window, &proto, &num_return)) { for (int i = 0; i < num_return; ++i) { - if (proto[i] == property->atom(otk::OBProperty::wm_delete_window)) { - _decorations |= Decor_Close; - _functions |= Func_Close; - if (frame) - frame->adjustSize(); // update the decorations - } else if (proto[i] == property->atom(otk::OBProperty::wm_take_focus)) + if (proto[i] == otk::Property::atoms.wm_delete_window) { + // this means we can request the window to close + _delete_window = true; + } else if (proto[i] == otk::Property::atoms.wm_take_focus) // if this protocol is requested, then the window will be notified // by the window manager whenever it receives focus _focus_notify = true; @@ -351,14 +411,13 @@ void OBClient::updateProtocols() } -void OBClient::updateNormalHints() +void Client::updateNormalHints() { XSizeHints size; long ret; int oldgravity = _gravity; // defaults - _gravity = NorthWestGravity; _size_inc.setPoint(1, 1); _base_size.setPoint(0, 0); _min_size.setPoint(0, 0); @@ -368,11 +427,21 @@ void OBClient::updateNormalHints() // point.. // get the hints from the window - if (XGetWMNormalHints(otk::OBDisplay::display, _window, &size, &ret)) { + if (XGetWMNormalHints(**otk::display, _window, &size, &ret)) { _positioned = (size.flags & (PPosition|USPosition)); - if (size.flags & PWinGravity) + if (size.flags & PWinGravity) { _gravity = size.win_gravity; + + // if the client has a frame, i.e. has already been mapped and is + // changing its gravity + if (frame && _gravity != oldgravity) { + // move our idea of the client's position based on its new gravity + int x, y; + frame->frameGravity(x, y); + _area.setPos(x, y); + } + } if (size.flags & PMinSize) _min_size.setPoint(size.min_width, size.min_height); @@ -386,32 +455,27 @@ void OBClient::updateNormalHints() if (size.flags & PResizeInc) _size_inc.setPoint(size.width_inc, size.height_inc); } - - // if the client has a frame, i.e. has already been mapped and is - // changing its gravity - if (frame && _gravity != oldgravity) { - // move our idea of the client's position based on its new gravity - int x, y; - frame->frameGravity(x, y); - _area.setPos(x, y); - } } -void OBClient::updateWMHints() +void Client::updateWMHints(bool initstate) { XWMHints *hints; // assume a window takes input if it doesnt specify _can_focus = true; - _urgent = false; + bool ur = false; - if ((hints = XGetWMHints(otk::OBDisplay::display, _window)) != NULL) { + if ((hints = XGetWMHints(**otk::display, _window)) != NULL) { if (hints->flags & InputHint) _can_focus = hints->input; + // only do this when initstate is true! + if (initstate && (hints->flags & StateHint)) + _iconic = hints->initial_state == IconicState; + if (hints->flags & XUrgencyHint) - _urgent = true; + ur = true; if (hints->flags & WindowGroupHint) { if (hints->window_group != _group) { @@ -424,21 +488,31 @@ void OBClient::updateWMHints() XFree(hints); } + + if (ur != _urgent) { + _urgent = ur; +#ifdef DEBUG + printf("DEBUG: Urgent Hint for 0x%lx: %s\n", + (long)_window, _urgent ? "ON" : "OFF"); +#endif + // fire the urgent callback if we're mapped, otherwise, wait until after + // we're mapped + if (_urgent && frame) + fireUrgent(); + } } -void OBClient::updateTitle() +void Client::updateTitle() { - const otk::OBProperty *property = Openbox::instance->property(); - _title = ""; // try netwm - if (! property->get(_window, otk::OBProperty::net_wm_name, - otk::OBProperty::utf8, &_title)) { + if (!otk::Property::get(_window, otk::Property::atoms.net_wm_name, + otk::Property::utf8, &_title)) { // try old x stuff - property->get(_window, otk::OBProperty::wm_name, - otk::OBProperty::ascii, &_title); + otk::Property::get(_window, otk::Property::atoms.wm_name, + otk::Property::ascii, &_title); } if (_title.empty()) @@ -449,18 +523,16 @@ void OBClient::updateTitle() } -void OBClient::updateIconTitle() +void Client::updateIconTitle() { - const otk::OBProperty *property = Openbox::instance->property(); - _icon_title = ""; // try netwm - if (! property->get(_window, otk::OBProperty::net_wm_icon_name, - otk::OBProperty::utf8, &_icon_title)) { + if (!otk::Property::get(_window, otk::Property::atoms.net_wm_icon_name, + otk::Property::utf8, &_icon_title)) { // try old x stuff - property->get(_window, otk::OBProperty::wm_icon_name, - otk::OBProperty::ascii, &_icon_title); + otk::Property::get(_window, otk::Property::atoms.wm_icon_name, + otk::Property::ascii, &_icon_title); } if (_title.empty()) @@ -468,38 +540,96 @@ void OBClient::updateIconTitle() } -void OBClient::updateClass() +void Client::updateClass() { - const otk::OBProperty *property = Openbox::instance->property(); - // set the defaults - _app_name = _app_class = ""; + _app_name = _app_class = _role = ""; - otk::OBProperty::StringVect v; + otk::Property::StringVect v; unsigned long num = 2; - if (! property->get(_window, otk::OBProperty::wm_class, - otk::OBProperty::ascii, &num, &v)) + if (otk::Property::get(_window, otk::Property::atoms.wm_class, + otk::Property::ascii, &num, &v)) { + if (num > 0) _app_name = v[0].c_str(); + if (num > 1) _app_class = v[1].c_str(); + } + + v.clear(); + num = 1; + if (otk::Property::get(_window, otk::Property::atoms.wm_window_role, + otk::Property::ascii, &num, &v)) { + if (num > 0) _role = v[0].c_str(); + } +} + + +void Client::updateStrut() +{ + unsigned long num = 4; + unsigned long *data; + if (!otk::Property::get(_window, otk::Property::atoms.net_wm_strut, + otk::Property::atoms.cardinal, &num, &data)) return; - if (num > 0) _app_name = v[0]; - if (num > 1) _app_class = v[1]; + if (num == 4) { + _strut.left = data[0]; + _strut.right = data[1]; + _strut.top = data[2]; + _strut.bottom = data[3]; + + openbox->screen(_screen)->updateStrut(); + } + + delete [] data; } -void OBClient::propertyHandler(const XPropertyEvent &e) +void Client::updateTransientFor() { - otk::OtkEventHandler::propertyHandler(e); - - const otk::OBProperty *property = Openbox::instance->property(); + Window t = 0; + Client *c = 0; + + if (XGetTransientForHint(**otk::display, _window, &t) && + t != _window) { // cant be transient to itself! + c = openbox->findClient(t); + assert(c != this); // if this happens then we need to check for it + + if (!c /*XXX: && _group*/) { + // not transient to a client, see if it is transient for a group + if (//t == _group->leader() || + t == None || + t == otk::display->screenInfo(_screen)->rootWindow()) { + // window is a transient for its group! + // XXX: for now this is treated as non-transient. + // this needs to be fixed! + } + } + } + + // if anything has changed... + if (c != _transient_for) { + if (_transient_for) + _transient_for->_transients.remove(this); // remove from old parent + _transient_for = c; + if (_transient_for) + _transient_for->_transients.push_back(this); // add to new parent + // XXX: change decor status? + } +} + + +void Client::propertyHandler(const XPropertyEvent &e) +{ + otk::EventHandler::propertyHandler(e); + // compress changes to a single property into a single change XEvent ce; - while (XCheckTypedEvent(otk::OBDisplay::display, e.type, &ce)) { + while (XCheckTypedEvent(**otk::display, e.type, &ce)) { // XXX: it would be nice to compress ALL changes to a property, not just // changes in a row without other props between. if (ce.xproperty.atom != e.atom) { - XPutBackEvent(otk::OBDisplay::display, &ce); + XPutBackEvent(**otk::display, &ce); break; } } @@ -508,52 +638,97 @@ void OBClient::propertyHandler(const XPropertyEvent &e) updateNormalHints(); else if (e.atom == XA_WM_HINTS) updateWMHints(); - else if (e.atom == property->atom(otk::OBProperty::net_wm_name) || - e.atom == property->atom(otk::OBProperty::wm_name)) + else if (e.atom == XA_WM_TRANSIENT_FOR) { + updateTransientFor(); + getType(); + calcLayer(); // type may have changed, so update the layer + setupDecorAndFunctions(); + frame->adjustSize(); // this updates the frame for any new decor settings + } + else if (e.atom == otk::Property::atoms.net_wm_name || + e.atom == otk::Property::atoms.wm_name) updateTitle(); - else if (e.atom == property->atom(otk::OBProperty::net_wm_icon_name) || - e.atom == property->atom(otk::OBProperty::wm_icon_name)) + else if (e.atom == otk::Property::atoms.net_wm_icon_name || + e.atom == otk::Property::atoms.wm_icon_name) updateIconTitle(); - else if (e.atom == property->atom(otk::OBProperty::wm_class)) + else if (e.atom == otk::Property::atoms.wm_class) updateClass(); - else if (e.atom == property->atom(otk::OBProperty::wm_protocols)) + else if (e.atom == otk::Property::atoms.wm_protocols) { updateProtocols(); - // XXX: transient for hint - // XXX: strut hint + setupDecorAndFunctions(); + frame->adjustSize(); // update the decorations + } + else if (e.atom == otk::Property::atoms.net_wm_strut) + updateStrut(); } -void OBClient::setWMState(long state) +void Client::setWMState(long state) { if (state == _wmstate) return; // no change switch (state) { case IconicState: - // XXX: cause it to iconify + setDesktop(ICONIC_DESKTOP); break; case NormalState: - // XXX: cause it to uniconify + setDesktop(openbox->screen(_screen)->desktop()); break; } - _wmstate = state; } -void OBClient::setDesktop(long target) +void Client::setDesktop(long target) { + if (target == _desktop) return; + printf("Setting desktop %ld\n", target); - assert(target >= 0 || target == (signed)0xffffffff); - //assert(target == 0xffffffff || target < MAX); - // XXX: move the window to the new desktop (and set root property) + if (!(target >= 0 || target == (signed)0xffffffff || + target == ICONIC_DESKTOP)) + return; + _desktop = target; + + // set the desktop hint, but not if we're iconifying + if (_desktop != ICONIC_DESKTOP) + otk::Property::set(_window, otk::Property::atoms.net_wm_desktop, + otk::Property::atoms.cardinal, (unsigned)_desktop); + + // 'move' the window to the new desktop + if (_desktop == openbox->screen(_screen)->desktop() || + _desktop == (signed)0xffffffff) + frame->show(); + else + frame->hide(); + + // Handle Iconic state. Iconic state is maintained by the client being a + // member of the ICONIC_DESKTOP, so this is where we make iconifying and + // uniconifying happen. + bool i = _desktop == ICONIC_DESKTOP; + if (i != _iconic) { // has the state changed? + _iconic = i; + if (_iconic) { + _wmstate = IconicState; + ignore_unmaps++; + // we unmap the client itself so that we can get MapRequest events, and + // because the ICCCM tells us to! + XUnmapWindow(**otk::display, _window); + } else { + _wmstate = NormalState; + XMapWindow(**otk::display, _window); + } + changeState(); + } + + frame->adjustState(); } -void OBClient::setState(StateAction action, long data1, long data2) +void Client::setState(StateAction action, long data1, long data2) { - const otk::OBProperty *property = Openbox::instance->property(); - bool restack = false, shadestate = _shaded; + bool shadestate = _shaded; + bool fsstate = _fullscreen; if (!(action == State_Add || action == State_Remove || action == State_Toggle)) @@ -566,127 +741,95 @@ void OBClient::setState(StateAction action, long data1, long data2) // if toggling, then pick whether we're adding or removing if (action == State_Toggle) { - if (state == property->atom(otk::OBProperty::net_wm_state_modal)) + if (state == otk::Property::atoms.net_wm_state_modal) action = _modal ? State_Remove : State_Add; - else if (state == - property->atom(otk::OBProperty::net_wm_state_maximized_vert)) + else if (state == otk::Property::atoms.net_wm_state_maximized_vert) action = _max_vert ? State_Remove : State_Add; - else if (state == - property->atom(otk::OBProperty::net_wm_state_maximized_horz)) + else if (state == otk::Property::atoms.net_wm_state_maximized_horz) action = _max_horz ? State_Remove : State_Add; - else if (state == property->atom(otk::OBProperty::net_wm_state_shaded)) + else if (state == otk::Property::atoms.net_wm_state_shaded) action = _shaded ? State_Remove : State_Add; - else if (state == - property->atom(otk::OBProperty::net_wm_state_skip_taskbar)) + else if (state == otk::Property::atoms.net_wm_state_skip_taskbar) action = _skip_taskbar ? State_Remove : State_Add; - else if (state == - property->atom(otk::OBProperty::net_wm_state_skip_pager)) + else if (state == otk::Property::atoms.net_wm_state_skip_pager) action = _skip_pager ? State_Remove : State_Add; - else if (state == - property->atom(otk::OBProperty::net_wm_state_fullscreen)) + else if (state == otk::Property::atoms.net_wm_state_fullscreen) action = _fullscreen ? State_Remove : State_Add; - else if (state == property->atom(otk::OBProperty::net_wm_state_above)) + else if (state == otk::Property::atoms.net_wm_state_above) action = _above ? State_Remove : State_Add; - else if (state == property->atom(otk::OBProperty::net_wm_state_below)) + else if (state == otk::Property::atoms.net_wm_state_below) action = _below ? State_Remove : State_Add; } if (action == State_Add) { - if (state == property->atom(otk::OBProperty::net_wm_state_modal)) { + if (state == otk::Property::atoms.net_wm_state_modal) { if (_modal) continue; _modal = true; // XXX: give it focus if another window has focus that shouldnt now - } else if (state == - property->atom(otk::OBProperty::net_wm_state_maximized_vert)){ + } else if (state == otk::Property::atoms.net_wm_state_maximized_vert) { if (_max_vert) continue; _max_vert = true; // XXX: resize the window etc - } else if (state == - property->atom(otk::OBProperty::net_wm_state_maximized_horz)){ + } else if (state == otk::Property::atoms.net_wm_state_maximized_horz) { if (_max_horz) continue; _max_horz = true; // XXX: resize the window etc - } else if (state == - property->atom(otk::OBProperty::net_wm_state_shaded)) { - if (_shaded) continue; - // shade when we're all thru here + } else if (state == otk::Property::atoms.net_wm_state_shaded) { shadestate = true; - } else if (state == - property->atom(otk::OBProperty::net_wm_state_skip_taskbar)) { + } else if (state == otk::Property::atoms.net_wm_state_skip_taskbar) { _skip_taskbar = true; - } else if (state == - property->atom(otk::OBProperty::net_wm_state_skip_pager)) { + } else if (state == otk::Property::atoms.net_wm_state_skip_pager) { _skip_pager = true; - } else if (state == - property->atom(otk::OBProperty::net_wm_state_fullscreen)) { - if (_fullscreen) continue; - _fullscreen = true; - restack = false; - } else if (state == - property->atom(otk::OBProperty::net_wm_state_above)) { + } else if (state == otk::Property::atoms.net_wm_state_fullscreen) { + fsstate = true; + } else if (state == otk::Property::atoms.net_wm_state_above) { if (_above) continue; _above = true; - restack = true; - } else if (state == - property->atom(otk::OBProperty::net_wm_state_below)) { + } else if (state == otk::Property::atoms.net_wm_state_below) { if (_below) continue; _below = true; - restack = true; } } else { // action == State_Remove - if (state == property->atom(otk::OBProperty::net_wm_state_modal)) { + if (state == otk::Property::atoms.net_wm_state_modal) { if (!_modal) continue; _modal = false; - } else if (state == - property->atom(otk::OBProperty::net_wm_state_maximized_vert)){ + } else if (state == otk::Property::atoms.net_wm_state_maximized_vert) { if (!_max_vert) continue; _max_vert = false; // XXX: resize the window etc - } else if (state == - property->atom(otk::OBProperty::net_wm_state_maximized_horz)){ + } else if (state == otk::Property::atoms.net_wm_state_maximized_horz) { if (!_max_horz) continue; _max_horz = false; // XXX: resize the window etc - } else if (state == - property->atom(otk::OBProperty::net_wm_state_shaded)) { - if (!_shaded) continue; - // unshade when we're all thru here + } else if (state == otk::Property::atoms.net_wm_state_shaded) { shadestate = false; - } else if (state == - property->atom(otk::OBProperty::net_wm_state_skip_taskbar)) { + } else if (state == otk::Property::atoms.net_wm_state_skip_taskbar) { _skip_taskbar = false; - } else if (state == - property->atom(otk::OBProperty::net_wm_state_skip_pager)) { + } else if (state == otk::Property::atoms.net_wm_state_skip_pager) { _skip_pager = false; - } else if (state == - property->atom(otk::OBProperty::net_wm_state_fullscreen)) { - if (!_fullscreen) continue; - _fullscreen = false; - restack = true; - } else if (state == - property->atom(otk::OBProperty::net_wm_state_above)) { + } else if (state == otk::Property::atoms.net_wm_state_fullscreen) { + fsstate = false; + } else if (state == otk::Property::atoms.net_wm_state_above) { if (!_above) continue; _above = false; - restack = true; - } else if (state == - property->atom(otk::OBProperty::net_wm_state_below)) { + } else if (state == otk::Property::atoms.net_wm_state_below) { if (!_below) continue; _below = false; - restack = true; } } } + // change fullscreen state before shading, as it will affect if the window + // can shade or not + if (fsstate != _fullscreen) + fullscreen(fsstate); if (shadestate != _shaded) shade(shadestate); - if (restack) { - calcLayer(); - Openbox::instance->screen(_screen)->restack(true, this); // raise - } + calcLayer(); } -void OBClient::toggleClientBorder(bool addborder) +void Client::toggleClientBorder(bool addborder) { // adjust our idea of where the client is, based on its border. When the // border is removed, the client should now be considered to be in a @@ -695,6 +838,7 @@ void OBClient::toggleClientBorder(bool addborder) // reversed. int x = _area.x(), y = _area.y(); switch(_gravity) { + default: case NorthWestGravity: case WestGravity: case SouthWestGravity: @@ -705,8 +849,17 @@ void OBClient::toggleClientBorder(bool addborder) if (addborder) x -= _border_width * 2; else x += _border_width * 2; break; + case NorthGravity: + case SouthGravity: + case CenterGravity: + case ForgetGravity: + case StaticGravity: + if (addborder) x -= _border_width; + else x += _border_width; + break; } switch(_gravity) { + default: case NorthWestGravity: case NorthGravity: case NorthEastGravity: @@ -717,39 +870,42 @@ void OBClient::toggleClientBorder(bool addborder) if (addborder) y -= _border_width * 2; else y += _border_width * 2; break; - default: - // no change for StaticGravity etc. + case WestGravity: + case EastGravity: + case CenterGravity: + case ForgetGravity: + case StaticGravity: + if (addborder) y -= _border_width; + else y += _border_width; break; } _area.setPos(x, y); if (addborder) { - XSetWindowBorderWidth(otk::OBDisplay::display, _window, _border_width); + XSetWindowBorderWidth(**otk::display, _window, _border_width); // move the client so it is back it the right spot _with_ its border! - XMoveWindow(otk::OBDisplay::display, _window, x, y); + XMoveWindow(**otk::display, _window, x, y); } else - XSetWindowBorderWidth(otk::OBDisplay::display, _window, 0); + XSetWindowBorderWidth(**otk::display, _window, 0); } -void OBClient::clientMessageHandler(const XClientMessageEvent &e) +void Client::clientMessageHandler(const XClientMessageEvent &e) { - otk::OtkEventHandler::clientMessageHandler(e); + otk::EventHandler::clientMessageHandler(e); if (e.format != 32) return; - const otk::OBProperty *property = Openbox::instance->property(); - - if (e.message_type == property->atom(otk::OBProperty::wm_change_state)) { + if (e.message_type == otk::Property::atoms.wm_change_state) { // compress changes into a single change bool compress = false; XEvent ce; - while (XCheckTypedEvent(otk::OBDisplay::display, e.type, &ce)) { + while (XCheckTypedEvent(**otk::display, e.type, &ce)) { // XXX: it would be nice to compress ALL messages of a type, not just // messages in a row without other message types between. if (ce.xclient.message_type != e.message_type) { - XPutBackEvent(otk::OBDisplay::display, &ce); + XPutBackEvent(**otk::display, &ce); break; } compress = true; @@ -758,16 +914,15 @@ void OBClient::clientMessageHandler(const XClientMessageEvent &e) setWMState(ce.xclient.data.l[0]); // use the found event else setWMState(e.data.l[0]); // use the original event - } else if (e.message_type == - property->atom(otk::OBProperty::net_wm_desktop)) { + } else if (e.message_type == otk::Property::atoms.net_wm_desktop) { // compress changes into a single change bool compress = false; XEvent ce; - while (XCheckTypedEvent(otk::OBDisplay::display, e.type, &ce)) { + while (XCheckTypedEvent(**otk::display, e.type, &ce)) { // XXX: it would be nice to compress ALL messages of a type, not just // messages in a row without other message types between. if (ce.xclient.message_type != e.message_type) { - XPutBackEvent(otk::OBDisplay::display, &ce); + XPutBackEvent(**otk::display, &ce); break; } compress = true; @@ -776,33 +931,56 @@ void OBClient::clientMessageHandler(const XClientMessageEvent &e) setDesktop(e.data.l[0]); // use the found event else setDesktop(e.data.l[0]); // use the original event - } else if (e.message_type == property->atom(otk::OBProperty::net_wm_state)) { + } else if (e.message_type == otk::Property::atoms.net_wm_state) { // can't compress these +#ifdef DEBUG + printf("net_wm_state %s %ld %ld for 0x%lx\n", + (e.data.l[0] == 0 ? "Remove" : e.data.l[0] == 1 ? "Add" : + e.data.l[0] == 2 ? "Toggle" : "INVALID"), + e.data.l[1], e.data.l[2], _window); +#endif setState((StateAction)e.data.l[0], e.data.l[1], e.data.l[2]); - } else if (e.message_type == - property->atom(otk::OBProperty::net_close_window)) { + } else if (e.message_type == otk::Property::atoms.net_close_window) { +#ifdef DEBUG + printf("net_close_window for 0x%lx\n", _window); +#endif close(); - } else if (e.message_type == - property->atom(otk::OBProperty::net_active_window)) { + } else if (e.message_type == otk::Property::atoms.net_active_window) { +#ifdef DEBUG + printf("net_active_window for 0x%lx\n", _window); +#endif + if (_iconic) + setDesktop(openbox->screen(_screen)->desktop()); + if (_shaded) + shade(false); + // XXX: deiconify focus(); - Openbox::instance->screen(_screen)->restack(true, this); // raise - } else { + openbox->screen(_screen)->raiseWindow(this); } } #if defined(SHAPE) -void OBClient::shapeHandler(const XShapeEvent &e) +void Client::shapeHandler(const XShapeEvent &e) { - otk::OtkEventHandler::shapeHandler(e); - - _shaped = e.shaped; - frame->adjustShape(); + otk::EventHandler::shapeHandler(e); + + if (e.kind == ShapeBounding) { + _shaped = e.shaped; + frame->adjustShape(); + } } #endif -void OBClient::resize(Corner anchor, int w, int h, int x, int y) +void Client::resize(Corner anchor, int w, int h) +{ + if (!(_functions & Func_Resize)) return; + internal_resize(anchor, w, h); +} + + +void Client::internal_resize(Corner anchor, int w, int h, int x, int y) { w -= _base_size.x(); h -= _base_size.y(); @@ -826,6 +1004,10 @@ void OBClient::resize(Corner anchor, int w, int h, int x, int y) w /= _size_inc.x(); h /= _size_inc.y(); + // you cannot resize to nothing + if (w < 1) w = 1; + if (h < 1) h = 1; + // store the logical size _logical_size.setPoint(w, h); @@ -856,27 +1038,52 @@ void OBClient::resize(Corner anchor, int w, int h, int x, int y) _area.setSize(w, h); - XResizeWindow(otk::OBDisplay::display, _window, w, h); + XResizeWindow(**otk::display, _window, w, h); // resize the frame to match the request frame->adjustSize(); - move(x, y); + internal_move(x, y); } -void OBClient::move(int x, int y) +void Client::move(int x, int y) +{ + if (!(_functions & Func_Move)) return; + internal_move(x, y); +} + + +void Client::internal_move(int x, int y) { _area.setPos(x, y); // move the frame to be in the requested position - frame->adjustPosition(); + if (frame) { // this can be called while mapping, before frame exists + frame->adjustPosition(); + + // send synthetic configure notify (we don't need to if we aren't mapped + // yet) + XEvent event; + event.type = ConfigureNotify; + event.xconfigure.display = **otk::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); + } } -void OBClient::close() +void Client::close() { XEvent ce; - const otk::OBProperty *property = Openbox::instance->property(); if (!(_functions & Func_Close)) return; @@ -887,149 +1094,286 @@ void OBClient::close() // explicitly killed. ce.xclient.type = ClientMessage; - ce.xclient.message_type = property->atom(otk::OBProperty::wm_protocols); - ce.xclient.display = otk::OBDisplay::display; + ce.xclient.message_type = otk::Property::atoms.wm_protocols; + ce.xclient.display = **otk::display; ce.xclient.window = _window; ce.xclient.format = 32; - ce.xclient.data.l[0] = property->atom(otk::OBProperty::wm_delete_window); + ce.xclient.data.l[0] = otk::Property::atoms.wm_delete_window; ce.xclient.data.l[1] = CurrentTime; ce.xclient.data.l[2] = 0l; ce.xclient.data.l[3] = 0l; ce.xclient.data.l[4] = 0l; - XSendEvent(otk::OBDisplay::display, _window, False, NoEventMask, &ce); + XSendEvent(**otk::display, _window, false, NoEventMask, &ce); } -void OBClient::changeState() +void Client::changeState() { - const otk::OBProperty *property = Openbox::instance->property(); - unsigned long state[2]; state[0] = _wmstate; state[1] = None; - property->set(_window, otk::OBProperty::wm_state, otk::OBProperty::wm_state, - state, 2); + otk::Property::set(_window, otk::Property::atoms.wm_state, + otk::Property::atoms.wm_state, state, 2); Atom netstate[10]; int num = 0; if (_modal) - netstate[num++] = property->atom(otk::OBProperty::net_wm_state_modal); + netstate[num++] = otk::Property::atoms.net_wm_state_modal; if (_shaded) - netstate[num++] = property->atom(otk::OBProperty::net_wm_state_shaded); + netstate[num++] = otk::Property::atoms.net_wm_state_shaded; if (_iconic) - netstate[num++] = property->atom(otk::OBProperty::net_wm_state_hidden); + netstate[num++] = otk::Property::atoms.net_wm_state_hidden; if (_skip_taskbar) - netstate[num++] = - property->atom(otk::OBProperty::net_wm_state_skip_taskbar); + netstate[num++] = otk::Property::atoms.net_wm_state_skip_taskbar; if (_skip_pager) - netstate[num++] = property->atom(otk::OBProperty::net_wm_state_skip_pager); + netstate[num++] = otk::Property::atoms.net_wm_state_skip_pager; if (_fullscreen) - netstate[num++] = property->atom(otk::OBProperty::net_wm_state_fullscreen); + netstate[num++] = otk::Property::atoms.net_wm_state_fullscreen; if (_max_vert) - netstate[num++] = - property->atom(otk::OBProperty::net_wm_state_maximized_vert); + netstate[num++] = otk::Property::atoms.net_wm_state_maximized_vert; if (_max_horz) - netstate[num++] = - property->atom(otk::OBProperty::net_wm_state_maximized_horz); + netstate[num++] = otk::Property::atoms.net_wm_state_maximized_horz; if (_above) - netstate[num++] = property->atom(otk::OBProperty::net_wm_state_above); + netstate[num++] = otk::Property::atoms.net_wm_state_above; if (_below) - netstate[num++] = property->atom(otk::OBProperty::net_wm_state_below); - property->set(_window, otk::OBProperty::net_wm_state, - otk::OBProperty::Atom_Atom, netstate, num); + netstate[num++] = otk::Property::atoms.net_wm_state_below; + otk::Property::set(_window, otk::Property::atoms.net_wm_state, + otk::Property::atoms.atom, netstate, num); + + calcLayer(); + + if (frame) + frame->adjustState(); +} + + +void Client::changeAllowedActions(void) +{ + Atom actions[9]; + int num = 0; + + actions[num++] = otk::Property::atoms.net_wm_action_change_desktop; + + if (_functions & Func_Shade) + actions[num++] = otk::Property::atoms.net_wm_action_shade; + if (_functions & Func_Close) + actions[num++] = otk::Property::atoms.net_wm_action_close; + if (_functions & Func_Move) + actions[num++] = otk::Property::atoms.net_wm_action_move; + if (_functions & Func_Iconify) + actions[num++] = otk::Property::atoms.net_wm_action_minimize; + if (_functions & Func_Resize) + actions[num++] = otk::Property::atoms.net_wm_action_resize; + if (_functions & Func_Fullscreen) + actions[num++] = otk::Property::atoms.net_wm_action_fullscreen; + if (_functions & Func_Maximize) { + actions[num++] = otk::Property::atoms.net_wm_action_maximize_horz; + actions[num++] = otk::Property::atoms.net_wm_action_maximize_vert; + } + + otk::Property::set(_window, otk::Property::atoms.net_wm_allowed_actions, + otk::Property::atoms.atom, actions, num); +} + + +void Client::applyStartupState() +{ + // these are in a carefully crafted order.. + + if (_iconic) { + _iconic = false; + setDesktop(ICONIC_DESKTOP); + } + if (_fullscreen) { + _fullscreen = false; + fullscreen(true); + } + if (_shaded) { + _shaded = false; + shade(true); + } + if (_urgent) + fireUrgent(); + if (_max_vert); // XXX: incomplete + if (_max_horz); // XXX: incomplete + + if (_skip_taskbar); // nothing to do for this + if (_skip_pager); // nothing to do for this + if (_modal); // nothing to do for this + if (_above); // nothing to do for this + if (_below); // nothing to do for this +} + + +void Client::fireUrgent() +{ + // call the python UrgentWindow callbacks + EventData data(_screen, this, EventAction::UrgentWindow, 0); + openbox->bindings()->fireEvent(&data); } -void OBClient::shade(bool shade) + +void Client::shade(bool shade) { - if (shade == _shaded) return; // already done + if (!(_functions & Func_Shade) || // can't + _shaded == shade) return; // already done - _wmstate = shade ? IconicState : NormalState; + // when we're iconic, don't change the wmstate + if (!_iconic) + _wmstate = shade ? IconicState : NormalState; _shaded = shade; changeState(); frame->adjustSize(); } -bool OBClient::focus() +void Client::fullscreen(bool fs) +{ + static FunctionFlags saved_func; + static DecorationFlags saved_decor; + static otk::Rect saved_area; + static otk::Point saved_logical_size; + + if (!(_functions & Func_Fullscreen) || // can't + _fullscreen == fs) return; // already done + + _fullscreen = fs; + changeState(); // change the state hints on the client + + if (fs) { + // save the functions and remove them + saved_func = _functions; + _functions = _functions & (Func_Close | Func_Fullscreen | Func_Iconify); + // save the decorations and remove them + saved_decor = _decorations; + _decorations = 0; + // save the area and adjust it (we don't call internal resize here for + // constraints on the size, etc, we just make it fullscreen). + saved_area = _area; + const otk::ScreenInfo *info = otk::display->screenInfo(_screen); + _area.setRect(0, 0, info->width(), info->height()); + saved_logical_size = _logical_size; + _logical_size.setPoint((info->width() - _base_size.x()) / _size_inc.x(), + (info->height() - _base_size.y()) / _size_inc.y()); + } else { + _functions = saved_func; + _decorations = saved_decor; + _area = saved_area; + _logical_size = saved_logical_size; + } + + changeAllowedActions(); // based on the new _functions + + frame->adjustSize(); // drop/replace the decor's and resize + frame->adjustPosition(); // get (back) in position! + + // raise (back) into our stacking layer + openbox->screen(_screen)->raiseWindow(this); + + // try focus us when we go into fullscreen mode + if (fs) focus(); +} + + +bool Client::focus() { - if (!(_can_focus || _focus_notify) || _focused) 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; + + // do a check to see if the window has already been unmapped or destroyed + // do this intelligently while watching out for unmaps we've generated + // (ignore_unmaps > 0) + XEvent ev; + if (XCheckTypedWindowEvent(**otk::display, _window, DestroyNotify, &ev)) { + XPutBackEvent(**otk::display, &ev); + return false; + } + while (XCheckTypedWindowEvent(**otk::display, _window, UnmapNotify, &ev)) { + if (ignore_unmaps) { + unmapHandler(ev.xunmap); + } else { + XPutBackEvent(**otk::display, &ev); + return false; + } + } if (_can_focus) - XSetInputFocus(otk::OBDisplay::display, _window, RevertToNone, CurrentTime); + XSetInputFocus(**otk::display, _window, + RevertToNone, CurrentTime); if (_focus_notify) { XEvent ce; - const otk::OBProperty *property = Openbox::instance->property(); - ce.xclient.type = ClientMessage; - ce.xclient.message_type = property->atom(otk::OBProperty::wm_protocols); - ce.xclient.display = otk::OBDisplay::display; + ce.xclient.message_type = otk::Property::atoms.wm_protocols; + ce.xclient.display = **otk::display; ce.xclient.window = _window; ce.xclient.format = 32; - ce.xclient.data.l[0] = property->atom(otk::OBProperty::wm_take_focus); - ce.xclient.data.l[1] = Openbox::instance->lastTime(); + ce.xclient.data.l[0] = otk::Property::atoms.wm_take_focus; + ce.xclient.data.l[1] = openbox->lastTime(); ce.xclient.data.l[2] = 0l; ce.xclient.data.l[3] = 0l; ce.xclient.data.l[4] = 0l; - XSendEvent(otk::OBDisplay::display, _window, False, NoEventMask, &ce); + XSendEvent(**otk::display, _window, False, NoEventMask, &ce); } return true; } -void OBClient::unfocus() +void Client::unfocus() const { if (!_focused) return; - assert(Openbox::instance->focusedClient() == this); - Openbox::instance->setFocusedClient(0); + assert(openbox->focusedClient() == this); + openbox->setFocusedClient(0); } -void OBClient::focusHandler(const XFocusChangeEvent &e) +void Client::focusHandler(const XFocusChangeEvent &e) { #ifdef DEBUG - printf("FocusIn for 0x%lx\n", e.window); +// printf("FocusIn for 0x%lx\n", e.window); #endif // DEBUG - OtkEventHandler::focusHandler(e); + otk::EventHandler::focusHandler(e); frame->focus(); _focused = true; - Openbox::instance->setFocusedClient(this); + openbox->setFocusedClient(this); } -void OBClient::unfocusHandler(const XFocusChangeEvent &e) +void Client::unfocusHandler(const XFocusChangeEvent &e) { #ifdef DEBUG - printf("FocusOut for 0x%lx\n", e.window); +// printf("FocusOut for 0x%lx\n", e.window); #endif // DEBUG - OtkEventHandler::unfocusHandler(e); + otk::EventHandler::unfocusHandler(e); frame->unfocus(); _focused = false; - if (Openbox::instance->focusedClient() == this) { - printf("UNFOCUSED!\n"); - Openbox::instance->setFocusedClient(this); - } + if (openbox->focusedClient() == this) + openbox->setFocusedClient(0); } -void OBClient::configureRequestHandler(const XConfigureRequestEvent &e) +void Client::configureRequestHandler(const XConfigureRequestEvent &e) { #ifdef DEBUG printf("ConfigureRequest for 0x%lx\n", e.window); #endif // DEBUG - OtkEventHandler::configureRequestHandler(e); + otk::EventHandler::configureRequestHandler(e); - // XXX: if we are iconic (or shaded? (fvwm does that)) ignore the event + // if we are iconic (or shaded (fvwm does this)) ignore the event + if (_iconic || _shaded) return; if (e.value_mask & CWBorderWidth) _border_width = e.border_width; @@ -1060,64 +1404,67 @@ void OBClient::configureRequestHandler(const XConfigureRequestEvent &e) if (e.value_mask & (CWX | CWY)) { int x = (e.value_mask & CWX) ? e.x : _area.x(); int y = (e.value_mask & CWY) ? e.y : _area.y(); - resize(corner, w, h, x, y); + internal_resize(corner, w, h, x, y); } else // if JUST resizing... - resize(corner, w, h); + internal_resize(corner, w, h); } else if (e.value_mask & (CWX | CWY)) { // if JUST moving... int x = (e.value_mask & CWX) ? e.x : _area.x(); int y = (e.value_mask & CWY) ? e.y : _area.y(); - move(x, y); + internal_move(x, y); } if (e.value_mask & CWStackMode) { switch (e.detail) { case Below: case BottomIf: - // XXX: lower the window + openbox->screen(_screen)->lowerWindow(this); break; case Above: case TopIf: default: - // XXX: raise the window + openbox->screen(_screen)->raiseWindow(this); break; } } } -void OBClient::unmapHandler(const XUnmapEvent &e) +void Client::unmapHandler(const XUnmapEvent &e) { + if (ignore_unmaps) { #ifdef DEBUG - printf("UnmapNotify for 0x%lx\n", e.window); +// printf("Ignored UnmapNotify for 0x%lx (event 0x%lx)\n", e.window, e.event); #endif // DEBUG - - if (ignore_unmaps) { ignore_unmaps--; return; } - OtkEventHandler::unmapHandler(e); +#ifdef DEBUG + printf("UnmapNotify for 0x%lx\n", e.window); +#endif // DEBUG + + otk::EventHandler::unmapHandler(e); // this deletes us etc - Openbox::instance->screen(_screen)->unmanageWindow(this); + openbox->screen(_screen)->unmanageWindow(this); } -void OBClient::destroyHandler(const XDestroyWindowEvent &e) +void Client::destroyHandler(const XDestroyWindowEvent &e) { #ifdef DEBUG printf("DestroyNotify for 0x%lx\n", e.window); #endif // DEBUG - OtkEventHandler::destroyHandler(e); + otk::EventHandler::destroyHandler(e); // this deletes us etc - Openbox::instance->screen(_screen)->unmanageWindow(this); + openbox->screen(_screen)->unmanageWindow(this); } -void OBClient::reparentHandler(const XReparentEvent &e) +void Client::reparentHandler(const XReparentEvent &e) { // this is when the client is first taken captive in the frame if (e.parent == frame->plate()) return; @@ -1126,7 +1473,7 @@ void OBClient::reparentHandler(const XReparentEvent &e) printf("ReparentNotify for 0x%lx\n", e.window); #endif // DEBUG - OtkEventHandler::reparentHandler(e); + otk::EventHandler::reparentHandler(e); /* This event is quite rare and is usually handled in unmapHandler. @@ -1135,8 +1482,27 @@ void OBClient::reparentHandler(const XReparentEvent &e) 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::display, &ev); + // this deletes us etc - Openbox::instance->screen(_screen)->unmanageWindow(this); + openbox->screen(_screen)->unmanageWindow(this); +} + +void Client::mapRequestHandler(const XMapRequestEvent &e) +{ +#ifdef DEBUG + printf("MapRequest for already managed 0x%lx\n", e.window); +#endif // DEBUG + + assert(_iconic); // we shouldn't be able to get this unless we're iconic + + // move to the current desktop (uniconify) + setDesktop(openbox->screen(_screen)->desktop()); + // XXX: should we focus/raise the window? (basically a net_wm_active_window) } }