]> Dogcows Code - chaz/openbox/blobdiff - src/client.cc
hack so text shows up
[chaz/openbox] / src / client.cc
index 353bda69d04e81109262ccce70fad54fe60c0245..2e78cb7672ff439db78dce1eb6f0f3a219b332dd 100644 (file)
@@ -77,6 +77,7 @@ Client::Client(int screen, Window window)
   updateClass();
   updateStrut();
   updateIcons();
+  updateKwmIcon();
   
   // this makes sure that these windows appear on all desktops
   if (/*_type == Type_Dock ||*/ _type == Type_Desktop)
@@ -150,10 +151,15 @@ void Client::getDesktop()
 {
   // defaults to the current desktop
   _desktop = openbox->screen(_screen)->desktop();
+  unsigned int d;
 
   if (otk::Property::get(_window, otk::Property::atoms.net_wm_desktop,
                          otk::Property::atoms.cardinal,
-                         (long unsigned*)&_desktop)) {
+                         (long unsigned*)&d)) {
+    if (d >= openbox->screen(_screen)->numDesktops() &&
+        d != 0xffffffff)
+      d = openbox->screen(_screen)->numDesktops() - 1;
+    _desktop = d;
 #ifdef DEBUG
 //    printf("Window requested desktop: %ld\n", _desktop);
 #endif
@@ -166,7 +172,7 @@ void Client::getType()
   _type = (WindowType) -1;
   
   unsigned long *val;
-  unsigned long num = (unsigned) -1;
+  unsigned long num;
   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
@@ -371,7 +377,7 @@ void Client::getState()
     _iconic = _skip_taskbar = _skip_pager = false;
   
   unsigned long *state;
-  unsigned long num = (unsigned) -1;
+  unsigned long num;
   
   if (otk::Property::get(_window, otk::Property::atoms.net_wm_state,
                          otk::Property::atoms.atom, &num, &state)) {
@@ -402,7 +408,6 @@ void Client::getState()
   }
 }
 
-
 void Client::getShaped()
 {
   _shaped = false;
@@ -421,12 +426,40 @@ void Client::getShaped()
 #endif // SHAPE
 }
 
+Client *Client::searchFocusTree(Client *node, Client *skip)
+{
+  List::const_iterator it, end = node->_transients.end();
+  Client *ret;
+
+  for (it = node->_transients.begin(); it != end; ++it) {
+    if (*it == skip) continue; // circular?
+    if ((ret = searchModalTree(*it, skip))) return ret; // got one
+    if ((*it)->_focused) return *it; // got one
+  }
+  return 0;
+}
 
 void Client::calcLayer() {
   StackLayer l;
+  bool fs = false;
 
+  // are we fullscreen, or do we have a fullscreen transient parent?
+  Client *c = this;
+  while (c) {
+    if (c->_fullscreen) {
+      fs =true;
+      break;
+    }
+    c = c->_transient_for;
+  }
+  if (!fs && _fullscreen) {
+    // is one of our transients focused?
+    c = searchFocusTree(this, this);
+    if (c) fs = true;
+  }
+  
   if (_iconic) l = Layer_Icon;
-  else if (_fullscreen) l = Layer_Fullscreen;
+  else if (fs) l = Layer_Fullscreen;
   else if (_type == Type_Desktop) l = Layer_Desktop;
   else if (_type == Type_Dock) {
     if (!_below) l = Layer_Top;
@@ -448,7 +481,6 @@ void Client::calcLayer() {
   }
 }
 
-
 void Client::updateProtocols()
 {
   Atom *proto;
@@ -471,7 +503,6 @@ void Client::updateProtocols()
   }
 }
 
-
 void Client::updateNormalHints()
 {
   XSizeHints size;
@@ -522,7 +553,6 @@ void Client::updateNormalHints()
   }
 }
 
-
 void Client::updateWMHints(bool initstate)
 {
   XWMHints *hints;
@@ -551,13 +581,24 @@ void Client::updateWMHints(bool initstate)
     } else // no group!
       _group = None;
 
+    if (hints->flags & IconPixmapHint) {
+      updateKwmIcon(); // try get the kwm icon first, this is a fallback only
+      if (_pixmap_icon == None) {
+        _pixmap_icon = hints->icon_pixmap;
+        if (hints->flags & IconMaskHint)
+          _pixmap_icon_mask = hints->icon_mask;
+        else
+          _pixmap_icon_mask = None;
+      }
+    }
+
     XFree(hints);
   }
 
   if (ur != _urgent) {
     _urgent = ur;
 #ifdef DEBUG
-    printf("DEBUG: Urgent Hint for 0x%lx: %s\n",
+    printf("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
@@ -567,7 +608,6 @@ void Client::updateWMHints(bool initstate)
   }
 }
 
-
 void Client::updateTitle()
 {
   _title = "";
@@ -587,7 +627,6 @@ void Client::updateTitle()
     frame->adjustTitle();
 }
 
-
 void Client::updateIconTitle()
 {
   _icon_title = "";
@@ -604,7 +643,6 @@ void Client::updateIconTitle()
     _icon_title = _("Unnamed Window");
 }
 
-
 void Client::updateClass()
 {
   // set the defaults
@@ -627,7 +665,6 @@ void Client::updateClass()
   }
 }
 
-
 void Client::updateStrut()
 {
   unsigned long num = 4;
@@ -651,7 +688,6 @@ void Client::updateStrut()
   delete [] data;
 }
 
-
 void Client::updateTransientFor()
 {
   Window t = 0;
@@ -684,10 +720,9 @@ void Client::updateTransientFor()
   }
 }
 
-
 void Client::updateIcons()
 {
-  unsigned long num = (unsigned) -1;
+  unsigned long num;
   unsigned long *data;
   unsigned long w, h, i = 0;
 
@@ -698,7 +733,7 @@ void Client::updateIcons()
   _nicons = 0;
 
   if (otk::Property::get(_window, otk::Property::atoms.net_wm_icon,
-                         otk::Property::atoms.cardinal, &num, &data)) {
+                          otk::Property::atoms.cardinal, &num, &data)) {
     // figure out how man valid icons are in here
     while (num - i > 2) {
       w = data[i++];
@@ -709,7 +744,7 @@ void Client::updateIcons()
     }
 
     _icons = new Icon[_nicons];
-
+    
     // store the icons
     i = 0;
     for (int j = 0; j < _nicons; ++j) {
@@ -720,14 +755,11 @@ void Client::updateIcons()
       i += w * h;
       assert(i <= num);
     }
-    printf("i: %lu\n", i);
-    printf("bleffffffff\n");
-    
+
     delete [] data;
   }
 
   if (_nicons <= 0) {
-    // set the default icon(s) XXX load these from the py
     _nicons = 1;
     _icons = new Icon[1];
     _icons[i].w = 0;
@@ -740,6 +772,21 @@ void Client::updateIcons()
   if (frame) frame->adjustIcon();
 }
 
+void Client::updateKwmIcon()
+{
+  _pixmap_icon = _pixmap_icon_mask = None;
+
+  unsigned long num = 2;
+  Pixmap *data;
+  if (otk::Property::get(_window, otk::Property::atoms.kwm_win_icon,
+                         otk::Property::atoms.kwm_win_icon, &num, &data)) {
+    if (num == 2) {
+      _pixmap_icon = data[0];
+      _pixmap_icon_mask = data[1];
+    }
+    delete [] data;
+  }
+}
 
 void Client::propertyHandler(const XPropertyEvent &e)
 {
@@ -786,9 +833,10 @@ void Client::propertyHandler(const XPropertyEvent &e)
     updateStrut();
   else if (e.atom == otk::Property::atoms.net_wm_icon)
     updateIcons();
+  else if (e.atom == otk::Property::atoms.kwm_win_icon)
+    updateKwmIcon();
 }
 
-
 void Client::setWMState(long state)
 {
   if (state == _wmstate) return; // no change
@@ -803,7 +851,6 @@ void Client::setWMState(long state)
   }
 }
 
-
 void Client::setDesktop(unsigned int target)
 {
   if (target == _desktop) return;
@@ -824,7 +871,6 @@ void Client::setDesktop(unsigned int target)
   openbox->screen(_screen)->updateStruts();
 }
 
-
 void Client::showhide()
 {
   bool show;
@@ -840,16 +886,16 @@ void Client::showhide()
   else      frame->hide();
 }
 
-
-void Client::setState(StateAction action, long data1, long data2)
+void Client::setState(Atom action, long data1, long data2)
 {
   bool shadestate = _shaded;
   bool fsstate = _fullscreen;
   bool maxh = _max_horz;
   bool maxv = _max_vert;
 
-  if (!(action == State_Add || action == State_Remove ||
-        action == State_Toggle))
+  if (!(action == otk::Property::atoms.net_wm_state_add ||
+        action == otk::Property::atoms.net_wm_state_remove ||
+        action == otk::Property::atoms.net_wm_state_toggle))
     return; // an invalid action was passed to the client message, ignore it
 
   for (int i = 0; i < 2; ++i) {
@@ -858,28 +904,37 @@ void Client::setState(StateAction action, long data1, long data2)
     if (! state) continue;
 
     // if toggling, then pick whether we're adding or removing
-    if (action == State_Toggle) {
+    if (action == otk::Property::atoms.net_wm_state_toggle) {
       if (state == otk::Property::atoms.net_wm_state_modal)
-        action = _modal ? State_Remove : State_Add;
+        action = _modal ? otk::Property::atoms.net_wm_state_remove :
+                          otk::Property::atoms.net_wm_state_add;
       else if (state == otk::Property::atoms.net_wm_state_maximized_vert)
-        action = _max_vert ? State_Remove : State_Add;
+        action = _max_vert ? otk::Property::atoms.net_wm_state_remove :
+                             otk::Property::atoms.net_wm_state_add;
       else if (state == otk::Property::atoms.net_wm_state_maximized_horz)
-        action = _max_horz ? State_Remove : State_Add;
+        action = _max_horz ? otk::Property::atoms.net_wm_state_remove :
+                             otk::Property::atoms.net_wm_state_add;
       else if (state == otk::Property::atoms.net_wm_state_shaded)
-        action = _shaded ? State_Remove : State_Add;
+        action = _shaded ? otk::Property::atoms.net_wm_state_remove :
+                           otk::Property::atoms.net_wm_state_add;
       else if (state == otk::Property::atoms.net_wm_state_skip_taskbar)
-        action = _skip_taskbar ? State_Remove : State_Add;
+        action = _skip_taskbar ? otk::Property::atoms.net_wm_state_remove :
+                                 otk::Property::atoms.net_wm_state_add;
       else if (state == otk::Property::atoms.net_wm_state_skip_pager)
-        action = _skip_pager ? State_Remove : State_Add;
+        action = _skip_pager ? otk::Property::atoms.net_wm_state_remove :
+                               otk::Property::atoms.net_wm_state_add;
       else if (state == otk::Property::atoms.net_wm_state_fullscreen)
-        action = _fullscreen ? State_Remove : State_Add;
+        action = _fullscreen ? otk::Property::atoms.net_wm_state_remove :
+                               otk::Property::atoms.net_wm_state_add;
       else if (state == otk::Property::atoms.net_wm_state_above)
-        action = _above ? State_Remove : State_Add;
+        action = _above ? otk::Property::atoms.net_wm_state_remove :
+                          otk::Property::atoms.net_wm_state_add;
       else if (state == otk::Property::atoms.net_wm_state_below)
-        action = _below ? State_Remove : State_Add;
+        action = _below ? otk::Property::atoms.net_wm_state_remove :
+                          otk::Property::atoms.net_wm_state_add;
     }
     
-    if (action == State_Add) {
+    if (action == otk::Property::atoms.net_wm_state_add) {
       if (state == otk::Property::atoms.net_wm_state_modal) {
         if (_modal) continue;
         _modal = true;
@@ -904,7 +959,7 @@ void Client::setState(StateAction action, long data1, long data2)
         _below = true;
       }
 
-    } else { // action == State_Remove
+    } else { // action == otk::Property::atoms.net_wm_state_remove
       if (state == otk::Property::atoms.net_wm_state_modal) {
         if (!_modal) continue;
         _modal = false;
@@ -954,7 +1009,6 @@ void Client::setState(StateAction action, long data1, long data2)
   changeState(); // change the hint to relect these changes
 }
 
-
 void Client::toggleClientBorder(bool addborder)
 {
   // adjust our idea of where the client is, based on its border. When the
@@ -1018,7 +1072,6 @@ void Client::toggleClientBorder(bool addborder)
     XSetWindowBorderWidth(**otk::display, _window, 0);
 }
 
-
 void Client::clientMessageHandler(const XClientMessageEvent &e)
 {
   otk::EventHandler::clientMessageHandler(e);
@@ -1070,7 +1123,7 @@ void Client::clientMessageHandler(const XClientMessageEvent &e)
             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]);
+    setState(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);
@@ -1102,10 +1155,17 @@ void Client::clientMessageHandler(const XClientMessageEvent &e)
     focus();
     if (e.data.l[1])
       openbox->screen(_screen)->raiseWindow(this);
+  } else if (e.message_type == otk::Property::atoms.openbox_restack_window) {
+#ifdef DEBUG
+    printf("openbox_restack_window for 0x%lx\n", _window);
+#endif
+    if (e.data.l[0] == 0)
+      openbox->screen(_screen)->raiseWindow(this);
+    else if (e.data.l[0] == 1)
+      openbox->screen(_screen)->lowerWindow(this);
   }
 }
 
-
 #if defined(SHAPE)
 void Client::shapeHandler(const XShapeEvent &e)
 {
@@ -1118,14 +1178,12 @@ void Client::shapeHandler(const XShapeEvent &e)
 }
 #endif
 
-
 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,
                              bool user, int x, int y)
 {
@@ -1225,21 +1283,20 @@ const Icon *Client::icon(const otk::Size &s) const
       li = i;
     }
   }
-  if (smallest == 0xffffffff) // didnt find one bigger than us...
-    return &_icons[li];
-  return &_icons[si];
+  if (largest == 0) // didnt find one smaller than the requested size
+    return &_icons[si];
+  return &_icons[li];
 }
 
-void Client::move(int x, int y)
+void Client::move(int x, int y, bool final)
 {
   if (!(_functions & Func_Move)) return;
   frame->frameGravity(x, y); // get the client's position based on x,y for the
                              // frame
-  internal_move(x, y);
+  internal_move(x, y, final);
 }
 
-
-void Client::internal_move(int x, int y)
+void Client::internal_move(int x, int y, bool final)
 {
   _area = otk::Rect(otk::Point(x, y), _area.size());
 
@@ -1249,32 +1306,33 @@ void Client::internal_move(int x, int y)
 
     // 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;
+    if (final) {
+      XEvent event;
+      event.type = ConfigureNotify;
+      event.xconfigure.display = **otk::display;
+      event.xconfigure.event = _window;
+      event.xconfigure.window = _window;
     
-    // root window coords with border in mind
-    event.xconfigure.x = x - _border_width + frame->size().left;
-    event.xconfigure.y = y - _border_width + frame->size().top;
+      // root window coords with border in mind
+      event.xconfigure.x = x - _border_width + frame->size().left;
+      event.xconfigure.y = y - _border_width + frame->size().top;
     
-    event.xconfigure.width = _area.width();
-    event.xconfigure.height = _area.height();
-    event.xconfigure.border_width = _border_width;
-    event.xconfigure.above = frame->plate();
-    event.xconfigure.override_redirect = False;
-    XSendEvent(event.xconfigure.display, event.xconfigure.window, False,
-               StructureNotifyMask, &event);
+      event.xconfigure.width = _area.width();
+      event.xconfigure.height = _area.height();
+      event.xconfigure.border_width = _border_width;
+      event.xconfigure.above = frame->plate();
+      event.xconfigure.override_redirect = False;
+      XSendEvent(event.xconfigure.display, event.xconfigure.window, False,
+                 StructureNotifyMask, &event);
 #if 0//def DEBUG
-    printf("Sent synthetic ConfigureNotify %d,%d %d,%d to 0x%lx\n",
-           event.xconfigure.x, event.xconfigure.y, event.xconfigure.width,
-           event.xconfigure.height, event.xconfigure.window);
+      printf("Sent synthetic ConfigureNotify %d,%d %d,%d to 0x%lx\n",
+             event.xconfigure.x, event.xconfigure.y, event.xconfigure.width,
+             event.xconfigure.height, event.xconfigure.window);
 #endif
+    }
   }
 }
 
-
 void Client::close()
 {
   XEvent ce;
@@ -1300,7 +1358,6 @@ void Client::close()
   XSendEvent(**otk::display, _window, false, NoEventMask, &ce);
 }
 
-
 void Client::changeState()
 {
   unsigned long state[2];
@@ -1340,7 +1397,6 @@ void Client::changeState()
     frame->adjustState();
 }
 
-
 void Client::changeAllowedActions(void)
 {
   Atom actions[9];
@@ -1384,7 +1440,6 @@ void Client::changeAllowedActions(void)
     else _max_vert = _max_horz = false;
 }
 
-
 void Client::remaximize()
 {
   int dir;
@@ -1400,7 +1455,6 @@ void Client::remaximize()
   maximize(true, dir, false);
 }
 
-
 void Client::applyStartupState()
 {
   // these are in a carefully crafted order..
@@ -1438,7 +1492,6 @@ void Client::applyStartupState()
   if (_below);        // nothing to do for this
 }
 
-
 void Client::fireUrgent()
 {
   // call the python UrgentWindow callbacks
@@ -1446,7 +1499,6 @@ void Client::fireUrgent()
   openbox->bindings()->fireEvent(&data);
 }
 
-
 void Client::shade(bool shade)
 {
   if (!(_functions & Func_Shade) || // can't
@@ -1460,7 +1512,6 @@ void Client::shade(bool shade)
   frame->adjustSize();
 }
 
-
 void Client::maximize(bool max, int dir, bool savearea)
 {
   assert(dir == 0 || dir == 1 || dir == 2);
@@ -1497,7 +1548,7 @@ void Client::maximize(bool max, int dir, bool savearea)
       if (otk::Property::get(_window, otk::Property::atoms.openbox_premax,
                              otk::Property::atoms.cardinal, &n,
                              (long unsigned**) &readdim)) {
-        if (n >= 4) {
+        if (n == 4) {
           if (_max_horz) {
             dimensions[0] = readdim[0];
             dimensions[2] = readdim[2];
@@ -1529,7 +1580,7 @@ void Client::maximize(bool max, int dir, bool savearea)
     if (otk::Property::get(_window, otk::Property::atoms.openbox_premax,
                            otk::Property::atoms.cardinal, &n,
                            (long unsigned**) &dimensions)) {
-      if (n >= 4) {
+      if (n == 4) {
         if (dir == 0 || dir == 1) { // horz
           x = (signed int)dimensions[0];
           w = (signed int)dimensions[2];
@@ -1567,7 +1618,6 @@ void Client::maximize(bool max, int dir, bool savearea)
   internal_resize(TopLeft, w, h, true, x, y);
 }
 
-
 void Client::fullscreen(bool fs, bool savearea)
 {
   static FunctionFlags saved_func;
@@ -1613,7 +1663,7 @@ void Client::fullscreen(bool fs, bool savearea)
     if (otk::Property::get(_window, otk::Property::atoms.openbox_premax,
                            otk::Property::atoms.cardinal, &n,
                            (long unsigned**) &dimensions)) {
-      if (n >= 4) {
+      if (n == 4) {
         x = dimensions[0];
         y = dimensions[1];
         w = dimensions[2];
@@ -1642,7 +1692,6 @@ void Client::fullscreen(bool fs, bool savearea)
   if (fs) focus();
 }
 
-
 void Client::iconify(bool iconic, bool curdesk)
 {
   if (_iconic == iconic) return; // nothing to do
@@ -1670,14 +1719,12 @@ void Client::iconify(bool iconic, bool curdesk)
   openbox->screen(_screen)->updateStruts();
 }
 
-
 void Client::disableDecorations(DecorationFlags flags)
 {
   _disabled_decorations = flags;
   setupDecorAndFunctions();
 }
 
-
 void Client::installColormap(bool install) const
 {
   XWindowAttributes wa;
@@ -1689,9 +1736,6 @@ void Client::installColormap(bool install) const
   }
 }
 
-
-// recursively searches the client 'tree' for a modal client, always skips the
-// topmost node (the window you're starting with)
 Client *Client::searchModalTree(Client *node, Client *skip)
 {
   List::const_iterator it, end = node->_transients.end();
@@ -1721,8 +1765,6 @@ bool Client::focus()
   // visible on the screen
   if (!(frame->visible() && (_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)
@@ -1766,8 +1808,6 @@ bool Client::focus()
 
 void Client::unfocus() const
 {
-  if (!_focused) return;
-
   assert(openbox->focusedClient() == this);
   openbox->setFocusedClient(0);
 }
@@ -1784,6 +1824,8 @@ void Client::focusHandler(const XFocusChangeEvent &e)
   _focused = true;
   frame->adjustFocus();
 
+  calcLayer(); // focus state can affect the stacking layer
+
   openbox->setFocusedClient(this);
 }
 
@@ -1799,6 +1841,8 @@ void Client::unfocusHandler(const XFocusChangeEvent &e)
   _focused = false;
   frame->adjustFocus();
 
+  calcLayer(); // focus state can affect the stacking layer
+
   if (openbox->focusedClient() == this)
     openbox->setFocusedClient(0);
 }
This page took 0.035534 seconds and 4 git commands to generate.