]> Dogcows Code - chaz/openbox/blobdiff - src/client.cc
use otk objects in the ob scripts by importing otk
[chaz/openbox] / src / client.cc
index a0dc6d76b159202ff0a7587f44a22689a891acf4..eeb09a374981a4fe5cfe9dc30d3f6386c6faa980 100644 (file)
@@ -7,14 +7,15 @@
 #include "client.hh"
 #include "frame.hh"
 #include "screen.hh"
-#include "bbscreen.hh"
 #include "openbox.hh"
+#include "bindings.hh"
 #include "otk/display.hh"
 #include "otk/property.hh"
 
 extern "C" {
 #include <X11/Xlib.h>
 #include <X11/Xutil.h>
+#include <X11/Xatom.h>
 
 #include <assert.h>
 
@@ -24,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);
@@ -36,173 +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
+  // we default to NormalState, visible
   _wmstate = NormalState;
-  // no default decors or functions, each has to be enabled
-  _decorations = _functions = 0;
+  // 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();
-  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();
-  // XXX: updateTransientFor();
+
+  // 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();
 
-/*
-#ifdef DEBUG
-  printf("Mapped window: 0x%lx\n"
-         "  title:         \t%s\t  icon title:    \t%s\n"
-         "  app name:      \t%s\t\t  class:         \t%s\n"
-         "  position:      \t%d, %d\t\t  size:          \t%d, %d\n"
-         "  desktop:       \t%lu\t\t  group:         \t0x%lx\n"
-         "  type:          \t%d\t\t  min size       \t%d, %d\n"
-         "  base size      \t%d, %d\t\t  max size       \t%d, %d\n"
-         "  size incr      \t%d, %d\t\t  gravity        \t%d\n"
-         "  wm state       \t%ld\t\t  can be focused:\t%s\n"
-         "  notify focus:  \t%s\t\t  urgent:        \t%s\n"
-         "  shaped:        \t%s\t\t  modal:         \t%s\n"
-         "  shaded:        \t%s\t\t  iconic:        \t%s\n"
-         "  vert maximized:\t%s\t\t  horz maximized:\t%s\n"
-         "  fullscreen:    \t%s\t\t  floating:      \t%s\n"
-         "  requested pos: \t%s\n",
-         _window,
-         _title.c_str(),
-         _icon_title.c_str(),
-         _app_name.c_str(),
-         _app_class.c_str(),
-         _area.x(), _area.y(),
-         _area.width(), _area.height(),
-         _desktop,
-         _group,
-         _type,
-         _min_x, _min_y,
-         _base_x, _base_y,
-         _max_x, _max_y,
-         _inc_x, _inc_y,
-         _gravity,
-         _wmstate,
-         _can_focus ? "yes" : "no",
-         _focus_notify ? "yes" : "no",
-         _urgent ? "yes" : "no",
-         _shaped ? "yes" : "no",
-         _modal ? "yes" : "no",
-         _shaded ? "yes" : "no",
-         _iconic ? "yes" : "no",
-         _max_vert ? "yes" : "no",
-         _max_horz ? "yes" : "no",
-         _fullscreen ? "yes" : "no",
-         _floating ? "yes" : "no",
-         _positioned ? "yes" : "no");
-#endif
-*/
+  // 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();
-
-  // 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);
+  // 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
+    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;
   }
@@ -212,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);
@@ -289,32 +304,37 @@ void OBClient::getArea()
 }
 
 
-void OBClient::getState()
+void Client::getState()
 {
-  const otk::OBProperty *property = Openbox::instance->property();
-
-  _modal = _shaded = _max_horz = _max_vert = _fullscreen = _floating = false;
+  _modal = _shaded = _max_horz = _max_vert = _fullscreen = _above = _below =
+    _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_fullscreen))
+      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] == otk::Property::atoms.net_wm_state_skip_pager)
+        _skip_pager = true;
+      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] == otk::Property::atoms.net_wm_state_above)
+        _above = true;
+      else if (state[i] == otk::Property::atoms.net_wm_state_below)
+        _below = true;
     }
 
     delete [] state;
@@ -322,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);
   }
@@ -341,24 +361,47 @@ void OBClient::getShaped()
 }
 
 
-void OBClient::updateProtocols()
-{
-  const otk::OBProperty *property = Openbox::instance->property();
+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 Client::updateProtocols()
+{
   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;
-        // XXX: update the decor?
-      } 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;
@@ -368,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);
@@ -385,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);
@@ -403,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) {
@@ -441,40 +488,51 @@ 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())
     _title = _("Unnamed Window");
+
+  if (frame)
+    frame->setTitle(_title);
 }
 
 
-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())
@@ -482,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;
     }
   }
@@ -522,50 +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)
 {
-  assert(target >= 0);
-  //assert(target == 0xffffffff || target < MAX);
+  if (target == _desktop) return;
+  
+  printf("Setting desktop %ld\n", target);
+
+  if (!(target >= 0 || target == (signed)0xffffffff ||
+        target == ICONIC_DESKTOP))
+    return;
   
-  // XXX: move the window to the new desktop
   _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 shadestate = _shaded;
+  bool fsstate = _fullscreen;
 
   if (!(action == State_Add || action == State_Remove ||
         action == State_Toggle))
@@ -578,91 +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_fullscreen))
+      else if (state == otk::Property::atoms.net_wm_state_skip_taskbar)
+        action = _skip_taskbar ? State_Remove : State_Add;
+      else if (state == otk::Property::atoms.net_wm_state_skip_pager)
+        action = _skip_pager ? State_Remove : State_Add;
+      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_floating))
-        action = _floating ? State_Remove : State_Add;
+      else if (state == otk::Property::atoms.net_wm_state_above)
+        action = _above ? State_Remove : State_Add;
+      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;
-        _shaded = true;
-        // XXX: hide the client window
-      } else if (state ==
-                 property->atom(otk::OBProperty::net_wm_state_fullscreen)) {
-        if (_fullscreen) continue;
-        _fullscreen = true;
-        // XXX: raise the window n shit
-      } else if (state ==
-                 property->atom(otk::OBProperty::net_wm_state_floating)) {
-        if (_floating) continue;
-        _floating = true;
-        // XXX: raise the window n shit
+      } else if (state == otk::Property::atoms.net_wm_state_shaded) {
+        shadestate = true;
+      } else if (state == otk::Property::atoms.net_wm_state_skip_taskbar) {
+        _skip_taskbar = true;
+      } else if (state == otk::Property::atoms.net_wm_state_skip_pager) {
+        _skip_pager = true;
+      } 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;
+      } else if (state == otk::Property::atoms.net_wm_state_below) {
+        if (_below) continue;
+        _below = 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;
-        _shaded = false;
-        // XXX: show the client window
-      } else if (state ==
-                 property->atom(otk::OBProperty::net_wm_state_fullscreen)) {
-        if (!_fullscreen) continue;
-        _fullscreen = false;
-        // XXX: lower the window to its proper layer
-      } else if (state ==
-                 property->atom(otk::OBProperty::net_wm_state_floating)) {
-        if (!_floating) continue;
-        _floating = false;
-        // XXX: lower the window to its proper layer
+      } else if (state == otk::Property::atoms.net_wm_state_shaded) {
+        shadestate = false;
+      } else if (state == otk::Property::atoms.net_wm_state_skip_taskbar) {
+        _skip_taskbar = false;
+      } else if (state == otk::Property::atoms.net_wm_state_skip_pager) {
+        _skip_pager = false;
+      } 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;
+      } else if (state == otk::Property::atoms.net_wm_state_below) {
+        if (!_below) continue;
+        _below = false;
       }
     }
   }
+  // 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);
+  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
@@ -671,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:
@@ -681,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:
@@ -693,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;
@@ -734,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;
@@ -752,24 +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 == otk::Property::atoms.net_close_window) {
+#ifdef DEBUG
+    printf("net_close_window for 0x%lx\n", _window);
+#endif
+    close();
+  } 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->screen(_screen)->raiseWindow(this);
+  }
 }
 
 
-#if defined(SHAPE) || defined(DOXYGEN_IGNORE)
-void OBClient::shapeHandler(const XShapeEvent &e)
+#if defined(SHAPE)
+void Client::shapeHandler(const XShapeEvent &e)
 {
-  otk::OtkEventHandler::shapeHandler(e);
-  
-  _shaped = e.shaped;
+  otk::EventHandler::shapeHandler(e);
+
+  if (e.kind == ShapeBounding) {
+    _shaped = e.shaped;
+    frame->adjustShape();
+  }
 }
 #endif
 
 
-void OBClient::resize(Corner anchor, int w, int h)
+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();
@@ -793,6 +1004,10 @@ void OBClient::resize(Corner anchor, int w, int h)
   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);
 
@@ -802,49 +1017,368 @@ void OBClient::resize(Corner anchor, int w, int h)
   w += _base_size.x();
   h += _base_size.y();
 
-  int x = _area.x(), y = _area.y();  
-  switch (anchor) {
-  case TopLeft:
-    break;
-  case TopRight:
-    x -= w - _area.width();
-    break;
-  case BottomLeft:
-    y -= h - _area.height();
-    break;
-  case BottomRight:
-    x -= w - _area.width();
-    y -= h - _area.height();
-    break;
+  if (x == INT_MIN || y == INT_MIN) {
+    x = _area.x();
+    y = _area.y();
+    switch (anchor) {
+    case TopLeft:
+      break;
+    case TopRight:
+      x -= w - _area.width();
+      break;
+    case BottomLeft:
+      y -= h - _area.height();
+      break;
+    case BottomRight:
+      x -= w - _area.width();
+      y -= h - _area.height();
+      break;
+    }
   }
 
   _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 Client::close()
+{
+  XEvent ce;
+
+  if (!(_functions & Func_Close)) return;
+
+  // XXX: itd be cool to do timeouts and shit here for killing the client's
+  //      process off
+  // like... if the window is around after 5 seconds, then the close button
+  // turns a nice red, and if this function is called again, the client is
+  // explicitly killed.
+
+  ce.xclient.type = ClientMessage;
+  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] = 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::display, _window, false, NoEventMask, &ce);
+}
+
+
+void Client::changeState()
+{
+  unsigned long state[2];
+  state[0] = _wmstate;
+  state[1] = None;
+  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++] = otk::Property::atoms.net_wm_state_modal;
+  if (_shaded)
+    netstate[num++] = otk::Property::atoms.net_wm_state_shaded;
+  if (_iconic)
+    netstate[num++] = otk::Property::atoms.net_wm_state_hidden;
+  if (_skip_taskbar)
+    netstate[num++] = otk::Property::atoms.net_wm_state_skip_taskbar;
+  if (_skip_pager)
+    netstate[num++] = otk::Property::atoms.net_wm_state_skip_pager;
+  if (_fullscreen)
+    netstate[num++] = otk::Property::atoms.net_wm_state_fullscreen;
+  if (_max_vert)
+    netstate[num++] = otk::Property::atoms.net_wm_state_maximized_vert;
+  if (_max_horz)
+    netstate[num++] = otk::Property::atoms.net_wm_state_maximized_horz;
+  if (_above)
+    netstate[num++] = otk::Property::atoms.net_wm_state_above;
+  if (_below)
+    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 Client::shade(bool shade)
+{
+  if (!(_functions & Func_Shade) || // can't
+      _shaded == shade) return;     // already done
+
+  // when we're iconic, don't change the wmstate
+  if (!_iconic)
+    _wmstate = shade ? IconicState : NormalState;
+  _shaded = shade;
+  changeState();
+  frame->adjustSize();
+}
+
+
+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()
+{
+  // 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::display, _window,
+                   RevertToNone, CurrentTime);
+
+  if (_focus_notify) {
+    XEvent ce;
+    ce.xclient.type = ClientMessage;
+    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] = 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::display, _window, False, NoEventMask, &ce);
+  }
+
+  return true;
+}
+
+
+void Client::unfocus() const
+{
+  if (!_focused) return;
+
+  assert(openbox->focusedClient() == this);
+  openbox->setFocusedClient(0);
 }
 
 
-void OBClient::configureRequestHandler(const XConfigureRequestEvent &e)
+void Client::focusHandler(const XFocusChangeEvent &e)
 {
-  OtkEventHandler::configureRequestHandler(e);
+#ifdef    DEBUG
+//  printf("FocusIn for 0x%lx\n", e.window);
+#endif // DEBUG
+  
+  otk::EventHandler::focusHandler(e);
+
+  frame->focus();
+  _focused = true;
+
+  openbox->setFocusedClient(this);
+}
+
+
+void Client::unfocusHandler(const XFocusChangeEvent &e)
+{
+#ifdef    DEBUG
+//  printf("FocusOut for 0x%lx\n", e.window);
+#endif // DEBUG
+  
+  otk::EventHandler::unfocusHandler(e);
 
-  // XXX: if we are iconic (or shaded? (fvwm does that)) ignore the event
+  frame->unfocus();
+  _focused = false;
+
+  if (openbox->focusedClient() == this)
+    openbox->setFocusedClient(0);
+}
+
+
+void Client::configureRequestHandler(const XConfigureRequestEvent &e)
+{
+#ifdef    DEBUG
+  printf("ConfigureRequest for 0x%lx\n", e.window);
+#endif // DEBUG
+  
+  otk::EventHandler::configureRequestHandler(e);
+
+  // 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;
 
-    // resize, then move, as specified in the EWMH section 7.7
+  // resize, then move, as specified in the EWMH section 7.7
   if (e.value_mask & (CWWidth | CWHeight)) {
     int w = (e.value_mask & CWWidth) ? e.width : _area.width();
     int h = (e.value_mask & CWHeight) ? e.height : _area.height();
@@ -866,61 +1400,109 @@ void OBClient::configureRequestHandler(const XConfigureRequestEvent &e)
       corner = TopLeft;
     }
 
-    resize(corner, w, h);
-  }
-
-  if (e.value_mask & (CWX | CWY)) {
+    // if moving AND resizing ...
+    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();
+      internal_resize(corner, w, h, x, y);
+    } else // if JUST resizing...
+      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 Client::reparentHandler(const XReparentEvent &e)
+{
+  // this is when the client is first taken captive in the frame
+  if (e.parent == frame->plate()) return;
+
+#ifdef    DEBUG
+  printf("ReparentNotify for 0x%lx\n", e.window);
+#endif // DEBUG
+
+  otk::EventHandler::reparentHandler(e);
+
+  /*
+    This event is quite rare and is usually handled in unmapHandler.
+    However, if the window is unmapped when the reparent event occurs,
+    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::display, &ev);
+  
+  // this deletes us etc
+  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)
+}
+
 }
This page took 0.05729 seconds and 4 git commands to generate.