yah. fuck

[chaz/openbox] / plugins / focus.c

diff --git a/plugins/focus.c b/plugins/focus.c
index 6a805478bef673b1889a0ee148834bd69182fa48..85d4f04439cad63c9b1f458e30faf682a96739a2 100644 (file)
--- a/plugins/focus.c
+++ b/plugins/focus.c
@@ -8,10 +8,10 @@
 
 /* config options */
 static gboolean follow_mouse = TRUE;
-static gboolean warp_on_desk_switch = TRUE;
-static gboolean focus_new = FALSE;
+static gboolean warp_on_desk_switch = FALSE;
+static gboolean focus_new = TRUE;
 
-static int skip_enter = 0;
+/*static int skip_enter = 0;*/
 
 static gboolean focus_under_pointer()
 {
@@ -29,13 +29,35 @@ static gboolean focus_under_pointer()
                 break;
         }
         if (it != NULL) {
-            client_focus(it->data);
-            return TRUE;
+            return client_normal(it->data) && client_focus(it->data);
         }
     }
     return FALSE;
 }
 
+static void chew_enter_events()
+{
+    XEvent e;
+
+    /* XXX... not anymore
+       skip the next enter event from the desktop switch so focus
+       doesn't skip briefly to what was under the pointer */
+
+    /* kill all enter events from prior to the desktop switch, we
+       aren't interested in them if we have found our own target
+       to focus.
+       XXX this is rude to other plugins...can this be done
+       better? count the events in the queue? */
+    while (XCheckTypedEvent(ob_display, EnterNotify, &e));
+/*
+    {
+        XPutBackEvent(ob_display, &e);
+        g_message("skip");
+        ++skip_enter;
+    }
+*/
+}
+
 static void focus_fallback(gboolean switching_desks)
 {
     GList *it;
@@ -43,23 +65,9 @@ static void focus_fallback(gboolean switching_desks)
     for (it = focus_order[screen_desktop]; it != NULL; it = it->next)
         if (client_normal(it->data) && client_focus(it->data)) {
             if (switching_desks) {
-                XEvent e;
                 Client *c = it->data;
 
-                /* XXX... not anymore
-                   skip the next enter event from the desktop switch so focus
-                   doesn't skip briefly to what was under the pointer */
-
-                /* kill all enter events from prior to the desktop switch, we
-                   aren't interested in them if we have found our own target
-                   to focus.
-                   XXX this is rude to other plugins...can this be done
-                   better? count the events in the queue? */
-                while (XCheckTypedEvent(ob_display, EnterNotify, &e));
-/*                    XPutBackEvent(ob_display, &e);
-                    g_message("skip");
-                    ++skip_enter;
-                    }*/
+                chew_enter_events();
 
                 if (warp_on_desk_switch) {
                     /* I have to do this warp twice! Otherwise windows dont get
@@ -74,7 +82,20 @@ static void focus_fallback(gboolean switching_desks)
         }
 }
 
-static void events(ObEvent *e, void *foo)
+static void focus_desktop()
+{
+    GList *it;
+
+    for (it = g_list_last(stacking_list); it != NULL; it = it->prev) {
+        Client *client = it->data;
+        if (client->type == Type_Desktop && client->frame->visible)
+            if (client_focus(client))
+                break;
+    }
+    chew_enter_events();
+}
+
+static void event(ObEvent *e, void *foo)
 {
     switch (e->type) {
     case Event_Client_Mapped:
@@ -85,7 +106,7 @@ static void events(ObEvent *e, void *foo)
     case Event_Client_Unmapped:
         if (ob_state == State_Exiting) break;
 
-        if (e->data.c.client->focused)
+        if (client_focused(e->data.c.client))
             if (!follow_mouse || !focus_under_pointer())
                 focus_fallback(FALSE);
         break;
@@ -101,18 +122,25 @@ static void events(ObEvent *e, void *foo)
         focus_fallback(TRUE);
         break;
 
+    case Event_Ob_ShowDesktop:
+        if (!e->data.o.num[0]) { /* hiding the desktop, showing the clients */
+            if (!follow_mouse || !focus_under_pointer())
+                focus_fallback(TRUE);
+        } else /* hiding clients, showing the desktop */
+            focus_desktop();
+        break;
+
     case Event_X_EnterNotify:
-        if (skip_enter) {
+/*        if (skip_enter) {
             if (e->data.x.client != NULL)
                 g_message("skipped enter %lx", e->data.x.client->window);
             else
                 g_message("skipped enter 'root'");
             --skip_enter;
         }
-        else if (e->data.x.client != NULL && client_normal(e->data.x.client)) {
-            g_message("enter %lx", e->data.x.client->window);
+        else*/
+        if (e->data.x.client != NULL && client_normal(e->data.x.client))
             client_focus(e->data.x.client);
-        }
         break;
 
     default:
@@ -125,11 +153,12 @@ void plugin_startup()
     dispatch_register(Event_Client_Mapped | 
                       Event_Ob_Desktop | 
                       Event_Client_Unmapped |
-                      Event_X_EnterNotify,
-                      (EventHandler)events, NULL);
+                      Event_X_EnterNotify |
+                      Event_Ob_ShowDesktop,
+                      (EventHandler)event, NULL);
 }
 
 void plugin_shutdown()
 {
-    dispatch_register(0, (EventHandler)events, NULL);
+    dispatch_register(0, (EventHandler)event, NULL);
 }