X-Git-Url: https://git.brokenzipper.com/gitweb?a=blobdiff_plain;f=openbox%2Faction.c;h=332e100d45c4a15bdd4ed0363d1e6310ece2642a;hb=d9e08487dffd010f4434d24b98a3a571b2e5a600;hp=e16e177a1d01da9991732a980e7e373dca224060;hpb=0f544808b8372a6c8a2cbfae214f2803dd3c6170;p=chaz%2Fopenbox

diff --git a/openbox/action.c b/openbox/action.c
index e16e177a..332e100d 100644
--- a/openbox/action.c
+++ b/openbox/action.c
@@ -1,3 +1,21 @@
+/* -*- indent-tabs-mode: nil; tab-width: 4; c-basic-offset: 4; -*-
+
+   action.c for the Openbox window manager
+   Copyright (c) 2003        Ben Jansens
+
+   This program is free software; you can redistribute it and/or modify
+   it under the terms of the GNU General Public License as published by
+   the Free Software Foundation; either version 2 of the License, or
+   (at your option) any later version.
+
+   This program is distributed in the hope that it will be useful,
+   but WITHOUT ANY WARRANTY; without even the implied warranty of
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+   GNU General Public License for more details.
+
+   See the COPYING file for a copy of the GNU General Public License.
+*/
+
 #include "debug.h"
 #include "client.h"
 #include "focus.h"
@@ -10,11 +28,35 @@
 #include "openbox.h"
 #include "grab.h"
 #include "keyboard.h"
+#include "event.h"
+#include "config.h"
 
 #include <glib.h>
 
+inline void client_action_start(union ActionData *data)
+{
+    if (config_focus_follow)
+        if (data->any.context != OB_FRAME_CONTEXT_CLIENT && !data->any.button)
+            grab_pointer(TRUE, OB_CURSOR_NONE);
+}
+
+inline void client_action_end(union ActionData *data)
+{
+    if (config_focus_follow)
+        if (data->any.context != OB_FRAME_CONTEXT_CLIENT) {
+            if (!data->any.button) {
+                grab_pointer(FALSE, OB_CURSOR_NONE);
+            } else {
+                ObClient *c;
+
+                if ((c = client_under_pointer()))
+                    event_enter_client(c);
+            }
+        }
+}
+
 typedef struct ActionString {
-    char *name;
+    const gchar *name;
     void (*func)(union ActionData *);
     void (*setup)(ObAction **, ObUserAction uact);
 } ActionString;
@@ -686,7 +728,7 @@ ActionString actionstrings[] =
     }
 };
 
-ObAction *action_from_string(char *name, ObUserAction uact)
+ObAction *action_from_string(const gchar *name, ObUserAction uact)
 {
     ObAction *a = NULL;
     gboolean exist = FALSE;
@@ -723,7 +765,7 @@ ObAction *action_parse(ObParseInst *i, xmlDocPtr doc, xmlNodePtr node,
             if (act->func == action_execute || act->func == action_restart) {
                 if ((n = parse_find_node("execute", node->xmlChildrenNode))) {
                     gchar *s = parse_string(doc, n);
-                    act->data.execute.path = ob_expand_tilde(s);
+                    act->data.execute.path = parse_expand_tilde(s);
                     g_free(s);
                 }
             } else if (act->func == action_showmenu) {
@@ -766,28 +808,58 @@ ObAction *action_parse(ObParseInst *i, xmlDocPtr doc, xmlNodePtr node,
     return act;
 }
 
-void action_run_full(ObAction *a, struct _ObClient *c,
-                     ObFrameContext context,
+void action_run_list(GSList *acts, ObClient *c, ObFrameContext context,
                      guint state, guint button, gint x, gint y,
                      gboolean cancel, gboolean done)
 {
+    GSList *it;
+    ObAction *a;
+    gboolean inter = FALSE;
+
+    if (!acts)
+        return;
+
     if (x < 0 && y < 0)
         screen_pointer_pos(&x, &y);
 
-    a->data.any.c = c;
-    a->data.any.x = x;
-    a->data.any.y = y;
-
-    a->data.any.button = button;
+    if (grab_on_keyboard())
+        inter = TRUE;
+    else
+        for (it = acts; it; it = g_slist_next(it)) {
+            a = it->data;
+            if (a->data.any.interactive) {
+                inter = TRUE;
+                break;
+            }
+        }
 
-    if (a->data.any.interactive) {
-        a->data.inter.cancel = cancel;
-        a->data.inter.final = done;
-        if (!(cancel || done))
-            keyboard_interactive_grab(state, c, context, a);
+    if (!inter) {
+        /* sometimes when we execute another app as an action,
+           it won't work right unless we XUngrabKeyboard first,
+           even though we grabbed the key/button Asychronously.
+           e.g. "gnome-panel-control --main-menu" */
+        XUngrabKeyboard(ob_display, event_lasttime);
     }
 
-    a->func(&a->data);
+    for (it = acts; it; it = g_slist_next(it)) {
+        a = it->data;
+
+        a->data.any.c = c;
+        a->data.any.context = context;
+        a->data.any.x = x;
+        a->data.any.y = y;
+
+        a->data.any.button = button;
+
+        if (a->data.any.interactive) {
+            a->data.inter.cancel = cancel;
+            a->data.inter.final = done;
+            if (!(cancel || done))
+                keyboard_interactive_grab(state, c, a);
+        }
+
+        a->func(&a->data);
+    }
 }
 
 void action_execute(union ActionData *data)
@@ -801,6 +873,7 @@ void action_execute(union ActionData *data)
                 g_warning("failed to execute '%s': %s",
                           cmd, e->message);
             }
+            g_free(cmd);
         } else {
             g_warning("failed to convert '%s' from utf8", data->execute.path);
         }
@@ -854,27 +927,38 @@ void action_raiselower(union ActionData *data)
         }
     }
 
-    if (raise)
+    if (raise) {
+        client_action_start(data);
         stacking_raise(CLIENT_AS_WINDOW(c));
-    else
+        client_action_end(data);
+    } else {
+        client_action_start(data);
         stacking_lower(CLIENT_AS_WINDOW(c));
+        client_action_end(data);
+    }
 }
 
 void action_raise(union ActionData *data)
 {
-    if (data->client.any.c)
+    if (data->client.any.c) {
+        client_action_start(data);
         stacking_raise(CLIENT_AS_WINDOW(data->client.any.c));
+        client_action_end(data);
+    }
 }
 
 void action_unshaderaise(union ActionData *data)
 {
     if (data->client.any.c) {
         if (data->client.any.c->shaded) {
-            grab_pointer(TRUE, OB_CURSOR_NONE);
+            client_action_start(data);
             client_shade(data->client.any.c, FALSE);
-            grab_pointer(FALSE, OB_CURSOR_NONE);
-        } else
+            client_action_end(data);
+        } else {
+            client_action_start(data);
             stacking_raise(CLIENT_AS_WINDOW(data->client.any.c));
+            client_action_end(data);
+        }
     }
 }
 
@@ -884,17 +968,20 @@ void action_shadelower(union ActionData *data)
         if (data->client.any.c->shaded)
             stacking_lower(CLIENT_AS_WINDOW(data->client.any.c));
         else {
-            grab_pointer(TRUE, OB_CURSOR_NONE);
+            client_action_start(data);
             client_shade(data->client.any.c, TRUE);
-            grab_pointer(FALSE, OB_CURSOR_NONE);
+            client_action_end(data);
         }
     }
 }
 
 void action_lower(union ActionData *data)
 {
-    if (data->client.any.c)
+    if (data->client.any.c) {
+        client_action_start(data);
         stacking_lower(CLIENT_AS_WINDOW(data->client.any.c));
+        client_action_end(data);
+    }
 }
 
 void action_close(union ActionData *data)
@@ -912,27 +999,27 @@ void action_kill(union ActionData *data)
 void action_shade(union ActionData *data)
 {
     if (data->client.any.c) { 
-        grab_pointer(TRUE, OB_CURSOR_NONE);
+        client_action_start(data);
         client_shade(data->client.any.c, TRUE);
-        grab_pointer(FALSE, OB_CURSOR_NONE);
+        client_action_end(data);
     }
 }
 
 void action_unshade(union ActionData *data)
 {
     if (data->client.any.c) {
-        grab_pointer(TRUE, OB_CURSOR_NONE);
+        client_action_start(data);
         client_shade(data->client.any.c, FALSE);
-        grab_pointer(FALSE, OB_CURSOR_NONE);
+        client_action_end(data);
     }
 }
 
 void action_toggle_shade(union ActionData *data)
 {
     if (data->client.any.c) {
-        grab_pointer(TRUE, OB_CURSOR_NONE);
+        client_action_start(data);
         client_shade(data->client.any.c, !data->client.any.c->shaded);
-        grab_pointer(FALSE, OB_CURSOR_NONE);
+        client_action_end(data);
     }
 }
 
@@ -948,9 +1035,9 @@ void action_move_relative_horz(union ActionData *data)
 {
     ObClient *c = data->relative.any.c;
     if (c) {
-        grab_pointer(TRUE, OB_CURSOR_NONE);
+        client_action_start(data);
         client_move(c, c->area.x + data->relative.delta, c->area.y);
-        grab_pointer(FALSE, OB_CURSOR_NONE);
+        client_action_end(data);
     }
 }
 
@@ -958,9 +1045,9 @@ void action_move_relative_vert(union ActionData *data)
 {
     ObClient *c = data->relative.any.c;
     if (c) {
-        grab_pointer(TRUE, OB_CURSOR_NONE);
+        client_action_start(data);
         client_move(c, c->area.x, c->area.y + data->relative.delta);
-        grab_pointer(FALSE, OB_CURSOR_NONE);
+        client_action_end(data);
     }
 }
 
@@ -968,11 +1055,11 @@ void action_resize_relative_horz(union ActionData *data)
 {
     ObClient *c = data->relative.any.c;
     if (c) {
-        grab_pointer(TRUE, OB_CURSOR_NONE);
+        client_action_start(data);
         client_resize(c,
                       c->area.width + data->relative.delta * c->size_inc.width,
                       c->area.height);
-        grab_pointer(FALSE, OB_CURSOR_NONE);
+        client_action_end(data);
     }
 }
 
@@ -980,69 +1067,97 @@ void action_resize_relative_vert(union ActionData *data)
 {
     ObClient *c = data->relative.any.c;
     if (c && !c->shaded) {
-        grab_pointer(TRUE, OB_CURSOR_NONE);
+        client_action_start(data);
         client_resize(c, c->area.width, c->area.height +
                       data->relative.delta * c->size_inc.height);
-        grab_pointer(FALSE, OB_CURSOR_NONE);
+        client_action_end(data);
     }
 }
 
 void action_maximize_full(union ActionData *data)
 {
-    if (data->client.any.c)
+    if (data->client.any.c) {
+        client_action_start(data);
         client_maximize(data->client.any.c, TRUE, 0, TRUE);
+        client_action_end(data);
+    }
 }
 
 void action_unmaximize_full(union ActionData *data)
 {
-    if (data->client.any.c)
+    if (data->client.any.c) {
+        client_action_start(data);
         client_maximize(data->client.any.c, FALSE, 0, TRUE);
+        client_action_end(data);
+    }
 }
 
 void action_toggle_maximize_full(union ActionData *data)
 {
-    if (data->client.any.c)
+    if (data->client.any.c) {
+        client_action_start(data);
         client_maximize(data->client.any.c,
                         !(data->client.any.c->max_horz ||
                           data->client.any.c->max_vert),
                         0, TRUE);
+        client_action_end(data);
+    }
 }
 
 void action_maximize_horz(union ActionData *data)
 {
-    if (data->client.any.c)
+    if (data->client.any.c) {
+        client_action_start(data);
         client_maximize(data->client.any.c, TRUE, 1, TRUE);
+        client_action_end(data);
+    }
 }
 
 void action_unmaximize_horz(union ActionData *data)
 {
-    if (data->client.any.c)
+    if (data->client.any.c) {
+        client_action_start(data);
         client_maximize(data->client.any.c, FALSE, 1, TRUE);
+        client_action_end(data);
+    }
 }
 
 void action_toggle_maximize_horz(union ActionData *data)
 {
-    if (data->client.any.c)
+    if (data->client.any.c) {
+        client_action_start(data);
         client_maximize(data->client.any.c,
                         !data->client.any.c->max_horz, 1, TRUE);
+        client_action_end(data);
+    }
 }
 
 void action_maximize_vert(union ActionData *data)
 {
-    if (data->client.any.c)
+    if (data->client.any.c) {
+        client_action_start(data);
         client_maximize(data->client.any.c, TRUE, 2, TRUE);
+        client_action_end(data);
+    }
 }
 
 void action_unmaximize_vert(union ActionData *data)
 {
-    if (data->client.any.c)
+    if (data->client.any.c) {
+        client_action_start(data);
         client_maximize(data->client.any.c, FALSE, 2, TRUE);
+        client_action_end(data);
+    }
 }
 
 void action_toggle_maximize_vert(union ActionData *data)
 {
-    if (data->client.any.c)
-        client_maximize(data->client.any.c, !data->client.any.c->max_vert, 2, TRUE);
+    if (data->client.any.c) {
+        client_action_start(data);
+        client_maximize(data->client.any.c,
+                        !data->client.any.c->max_vert, 2, TRUE);
+        client_action_end(data);
+    }
 }
 
 void action_send_to_desktop(union ActionData *data)
@@ -1105,10 +1220,12 @@ void action_toggle_decorations(union ActionData *data)
 {
     ObClient *c = data->client.any.c;
 
-    if (!c) return;
-
-    c->decorate = !c->decorate;
-    client_setup_decor_and_functions(c);
+    if (c) {
+        client_action_start(data);
+        c->decorate = !c->decorate;
+        client_setup_decor_and_functions(c);
+        client_action_end(data);
+    }
 }
 
 static guint32 pick_corner(int x, int y, int cx, int cy, int cw, int ch)
@@ -1167,7 +1284,7 @@ void action_restart(union ActionData *data)
 
 void action_exit(union ActionData *data)
 {
-    ob_exit();
+    ob_exit(0);
 }
 
 void action_showmenu(union ActionData *data)
@@ -1222,9 +1339,9 @@ void action_movetoedge(union ActionData *data)
         g_assert_not_reached();
     }
     frame_frame_gravity(c->frame, &x, &y);
-    grab_pointer(TRUE, OB_CURSOR_NONE);
+    client_action_start(data);
     client_move(c, x, y);
-    grab_pointer(FALSE, OB_CURSOR_NONE);
+    client_action_end(data);
 
 }
 
@@ -1232,11 +1349,12 @@ void action_growtoedge(union ActionData *data)
 {
     int x, y, width, height, dest;
     ObClient *c = data->diraction.any.c;
-    Rect *a = screen_area(c->desktop);
+    Rect *a;
 
     if (!c)
         return;
     
+    a = screen_area(c->desktop);
     x = c->frame->area.x;
     y = c->frame->area.y;
     width = c->frame->area.width;
@@ -1287,9 +1405,9 @@ void action_growtoedge(union ActionData *data)
     frame_frame_gravity(c->frame, &x, &y);
     width -= c->frame->size.left + c->frame->size.right;
     height -= c->frame->size.top + c->frame->size.bottom;
-    grab_pointer(TRUE, OB_CURSOR_NONE);
+    client_action_start(data);
     client_move_resize(c, x, y, width, height);
-    grab_pointer(FALSE, OB_CURSOR_NONE);
+    client_action_end(data);
 }
 
 void action_send_to_layer(union ActionData *data)
@@ -1303,10 +1421,12 @@ void action_toggle_layer(union ActionData *data)
     ObClient *c = data->layer.any.c;
 
     if (c) {
+        client_action_start(data);
         if (data->layer.layer < 0)
             client_set_layer(c, c->below ? 0 : -1);
         else if (data->layer.layer > 0)
             client_set_layer(c, c->above ? 0 : 1);
+        client_action_end(data);
     }
 }