X-Git-Url: https://git.brokenzipper.com/gitweb?a=blobdiff_plain;f=util%2Fepist%2Fscreen.cc;h=8597e5953d2df2392a7c4aaea8f510cf866b6835;hb=6a8f5f44e123c9ade7964e2051dd0d6a9858961c;hp=d8d8edcaa5de748451d4cfeee39417b41bce42b0;hpb=c6234a4de0263828e89e3b8812cbb93e72d13ffc;p=chaz%2Fopenbox

diff --git a/util/epist/screen.cc b/util/epist/screen.cc
index d8d8edca..8597e595 100644
--- a/util/epist/screen.cc
+++ b/util/epist/screen.cc
@@ -70,7 +70,9 @@ screen::screen(epist *epist, int number) {
   }
  
   XSelectInput(_epist->getXDisplay(), _root, PropertyChangeMask);
-    
+
+  updateNumDesktops();
+  updateActiveDesktop();
   updateClientList();
   updateActiveWindow();
 }
@@ -121,6 +123,10 @@ void screen::processEvent(const XEvent &e) {
   switch (e.type) {
   case PropertyNotify:
     // root window
+    if (e.xproperty.atom == _xatom->getAtom(XAtom::net_number_of_desktops))
+      updateNumDesktops();
+    if (e.xproperty.atom == _xatom->getAtom(XAtom::net_current_desktop))
+      updateActiveDesktop();
     if (e.xproperty.atom == _xatom->getAtom(XAtom::net_active_window))
       updateActiveWindow();
     if (e.xproperty.atom == _xatom->getAtom(XAtom::net_client_list)) {
@@ -165,6 +171,30 @@ void screen::handleKeypress(const XEvent &e) {
         cycleWindow(false);
         return;
 
+      case Action::nextWindowOnAllWorkspaces:
+        cycleWindow(true, true);
+        return;
+
+      case Action::prevWindowOnAllWorkspaces:
+        cycleWindow(false, true);
+        return;
+
+      case Action::nextWindowOfClass:
+        cycleWindow(true, false, true);
+        return;
+
+      case Action::prevWindowOfClass:
+        cycleWindow(false, false, true);
+        return;
+
+      case Action::nextWindowOfClassOnAllWorkspaces:
+        cycleWindow(true, true, true);
+        return;
+
+      case Action::prevWindowOfClassOnAllWorkspaces:
+        cycleWindow(false, true, true);
+        return;
+
       case Action::changeWorkspace:
         changeWorkspace(it->number());
         return;
@@ -191,6 +221,17 @@ void screen::handleKeypress(const XEvent &e) {
           window->lower();
           return;
 
+        case Action::sendTo:
+          window->sendTo(it->number());
+          return;
+
+        case Action::toggleomnipresent:
+          if (window->desktop() == 0xffffffff)
+            window->sendTo(_active_desktop);
+          else
+            window->sendTo(0xffffffff);
+          return;
+
         case Action::toggleshade:
           window->shade(! window->shaded());
           return;
@@ -217,6 +258,25 @@ bool screen::doAddWindow(Window window) const {
 }
 
 
+void screen::updateNumDesktops() {
+  assert(_managed);
+
+  if (! _xatom->getValue(_root, XAtom::net_number_of_desktops, XAtom::cardinal,
+                         (unsigned long)_num_desktops))
+    _num_desktops = 1;  // assume that there is at least 1 desktop!
+}
+
+
+void screen::updateActiveDesktop() {
+  assert(_managed);
+
+  if (! _xatom->getValue(_root, XAtom::net_current_desktop, XAtom::cardinal,
+                         (unsigned long)_active_desktop))
+    _active_desktop = 0;  // there must be at least one desktop, and it must
+                          // be the current one
+}
+
+
 void screen::updateClientList() {
   assert(_managed);
 
@@ -299,7 +359,10 @@ void screen::updateActiveWindow() {
  */
 
 
-void screen::cycleWindow(const bool forward) const {
+void screen::cycleWindow(const bool forward, const bool alldesktops,
+                         const bool sameclass) const {
+  assert(_managed);
+
   if (_clients.empty()) return;
     
   WindowList::const_iterator target = _active;
@@ -317,42 +380,41 @@ void screen::cycleWindow(const bool forward) const {
         target = _clients.end();
       --target;
     }
-  } while (target == _clients.end() || (*target)->iconic());
+  } while (target == _clients.end() ||
+           (*target)->iconic() ||
+           (! alldesktops && (*target)->desktop() != _active_desktop) ||
+           (sameclass && _active != _clients.end() &&
+            (*target)->appClass() != (*_active)->appClass()));
   
-  if (target != _clients.end()) {
-    // we dont send an ACTIVE_WINDOW client message because that would also
-    // unshade the window if it was shaded
-    XSetInputFocus(_epist->getXDisplay(), (*target)->window(), RevertToNone,
-                   CurrentTime);
-    XRaiseWindow(_epist->getXDisplay(), (*target)->window());
-  }
+  if (target != _clients.end())
+    (*target)->focus();
 }
 
 
-void screen::cycleWorkspace(const bool forward) const {
-  unsigned long currentDesktop = 0;
-  unsigned long numDesktops = 0;
-  
-  if (_xatom->getValue(_root, XAtom::net_current_desktop, XAtom::cardinal,
-                       currentDesktop)) {
-    if (forward)     
-      ++currentDesktop;
-    else
-      --currentDesktop;
-
-    _xatom->getValue(_root, XAtom::net_number_of_desktops, XAtom::cardinal,
-                     numDesktops);
-    
-    if ( ( (signed)currentDesktop) == -1)
-      currentDesktop = numDesktops - 1;
-    else if (currentDesktop >= numDesktops)
-      currentDesktop = 0;
+void screen::cycleWorkspace(const bool forward, const bool loop) const {
+  assert(_managed);
 
-    changeWorkspace(currentDesktop);
+  unsigned int destination = _active_desktop;
+
+  if (forward) {
+    if (destination < _num_desktops - 1)
+      ++destination;
+    else if (loop)
+      destination = 0;
+  } else {
+    if (destination > 0)
+      --destination;
+    else if (loop)
+      destination = _num_desktops - 1;
   }
+
+  if (destination != _active_desktop) 
+    changeWorkspace(destination);
 }
 
 
 void screen::changeWorkspace(const int num) const {
+  assert(_managed);
+
   _xatom->sendClientMessage(_root, XAtom::net_current_desktop, _root, num);
 }