X-Git-Url: https://git.dogcows.com/gitweb?a=blobdiff_plain;f=openbox%2Factions%2Fdesktop.c;h=e1bd5520808f195f5e80e836ca26592c1aef5026;hb=ec328fc04fe6e8d8c3d1ee386d4964963eb2ee17;hp=4489f8122dfbbea9da5e2b02f2f90fa5577bb50a;hpb=74aa6fe7cce662817ea4bca516d24d6d8d1e9a9b;p=chaz%2Fopenbox

diff --git a/openbox/actions/desktop.c b/openbox/actions/desktop.c
index 4489f812..e1bd5520 100644
--- a/openbox/actions/desktop.c
+++ b/openbox/actions/desktop.c
@@ -3,55 +3,113 @@
 #include "openbox/client.h"
 #include <glib.h>
 
+typedef enum {
+    LAST,
+    RELATIVE,
+    ABSOLUTE
+} SwitchType;
+
 typedef struct {
-    gboolean last;
-    guint desktop;
+    SwitchType type;
+    union {
+        struct {
+            guint desktop;
+        } abs;
+
+        struct {
+            gboolean linear;
+            gboolean wrap;
+            ObDirection dir;
+        } rel;
+    };
     gboolean send;
     gboolean follow;
 } Options;
 
-static gpointer setup_func(ObParseInst *i, xmlDocPtr doc, xmlNodePtr node);
-static void     free_func(gpointer options);
+static gpointer setup_go_func(xmlNodePtr node);
+static gpointer setup_send_func(xmlNodePtr node);
 static gboolean run_func(ObActionsData *data, gpointer options);
 
-void action_desktop_startup()
+void action_desktop_startup(void)
 {
-    actions_register("Desktop",
-                     setup_func,
-                     free_func,
-                     run_func,
+    actions_register("GoToDesktop", setup_go_func, g_free, run_func,
+                     NULL, NULL);
+    actions_register("SendToDesktop", setup_send_func, g_free, run_func,
                      NULL, NULL);
 }
 
-static gpointer setup_func(ObParseInst *i, xmlDocPtr doc, xmlNodePtr node)
+static gpointer setup_go_func(xmlNodePtr node)
 {
     xmlNodePtr n;
     Options *o;
 
     o = g_new0(Options, 1);
-    o->follow = TRUE;
+    /* don't go anywhere if theres no options given */
+    o->type = ABSOLUTE;
+    o->abs.desktop = screen_desktop;
+    /* wrap by default - it's handy! */
+    o->rel.wrap = TRUE;
 
-    if ((n = parse_find_node("desktop", node))) {
-        gchar *s = parse_string(doc, n);
+    if ((n = obt_parse_find_node(node, "to"))) {
+        gchar *s = obt_parse_node_string(n);
         if (!g_ascii_strcasecmp(s, "last"))
-            o->last = TRUE;
-        else
-            o->desktop = parse_int(doc, n) - 1;
+            o->type = LAST;
+        else if (!g_ascii_strcasecmp(s, "next")) {
+            o->type = RELATIVE;
+            o->rel.linear = TRUE;
+            o->rel.dir = OB_DIRECTION_EAST;
+        }
+        else if (!g_ascii_strcasecmp(s, "previous")) {
+            o->type = RELATIVE;
+            o->rel.linear = TRUE;
+            o->rel.dir = OB_DIRECTION_WEST;
+        }
+        else if (!g_ascii_strcasecmp(s, "north") ||
+                 !g_ascii_strcasecmp(s, "up")) {
+            o->type = RELATIVE;
+            o->rel.dir = OB_DIRECTION_NORTH;
+        }
+        else if (!g_ascii_strcasecmp(s, "south") ||
+                 !g_ascii_strcasecmp(s, "down")) {
+            o->type = RELATIVE;
+            o->rel.dir = OB_DIRECTION_SOUTH;
+        }
+        else if (!g_ascii_strcasecmp(s, "west") ||
+                 !g_ascii_strcasecmp(s, "left")) {
+            o->type = RELATIVE;
+            o->rel.dir = OB_DIRECTION_WEST;
+        }
+        else if (!g_ascii_strcasecmp(s, "east") ||
+                 !g_ascii_strcasecmp(s, "right")) {
+            o->type = RELATIVE;
+            o->rel.dir = OB_DIRECTION_EAST;
+        }
+        else {
+            o->type = ABSOLUTE;
+            o->abs.desktop = atoi(s) - 1;
+        }
         g_free(s);
     }
-    if ((n = parse_find_node("send", node)))
-        o->send = parse_bool(doc, n);
-    if ((n = parse_find_node("follow", node)))
-        o->follow = parse_bool(doc, n);
+
+    if ((n = obt_parse_find_node(node, "wrap")))
+        o->rel.wrap = obt_parse_node_bool(n);
 
     return o;
 }
 
-static void free_func(gpointer options)
+static gpointer setup_send_func(xmlNodePtr node)
 {
-    Options *o = options;
+    xmlNodePtr n;
+    Options *o;
+
+    o = setup_go_func(node);
+    o->send = TRUE;
+    o->follow = TRUE;
 
-    g_free(o);
+    if ((n = obt_parse_find_node(node, "follow")))
+        o->follow = obt_parse_node_bool(n);
+
+    return o;
 }
 
 /* Always return FALSE because its not interactive */
@@ -60,21 +118,30 @@ static gboolean run_func(ObActionsData *data, gpointer options)
     Options *o = options;
     guint d;
 
-    if (o->last)
+    switch (o->type) {
+    case LAST:
         d = screen_last_desktop;
-    else
-        d = o->desktop;
+        break;
+    case ABSOLUTE:
+        d = o->abs.desktop;
+        break;
+    case RELATIVE:
+        d = screen_find_desktop(screen_desktop,
+                                o->rel.dir, o->rel.wrap, o->rel.linear);
+        break;
+    }
 
     if (d < screen_num_desktops && d != screen_desktop) {
-        gboolean go = !o->send;
-        if (o->send) {
-            if (data->client && client_normal(data->client)) {
-                client_set_desktop(data->client, d, o->follow, FALSE);
-                go = TRUE;
-            }
+        gboolean go = TRUE;
+
+        actions_client_move(data, TRUE);
+        if (o->send && data->client && client_normal(data->client)) {
+            client_set_desktop(data->client, d, o->follow, FALSE);
+            go = o->follow;
         }
-        if (go)
-            screen_set_desktop(d, TRUE);
+
+        if (go) screen_set_desktop(d, TRUE);
+        actions_client_move(data, FALSE);
     }
     return FALSE;
 }