From d846979db91a81781993b3e14f6bc0803b9c3ed1 Mon Sep 17 00:00:00 2001
From: Dana Jansens <danakj@orodu.net>
Date: Fri, 22 Jun 2007 18:51:28 +0000
Subject: [PATCH] add the decorations action

---
 Makefile.am                   |  1 +
 openbox/action.c              | 15 ---------
 openbox/actions/all.c         |  1 +
 openbox/actions/all.h         |  1 +
 openbox/actions/decorations.c | 62 +++++++++++++++++++++++++++++++++++
 5 files changed, 65 insertions(+), 15 deletions(-)
 create mode 100644 openbox/actions/decorations.c

diff --git a/Makefile.am b/Makefile.am
index 4d2b78a8..b067543c 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -160,6 +160,7 @@ openbox_openbox_SOURCES = \
 	openbox/actions/close.c \
 	openbox/actions/cyclewindows.c \
 	openbox/actions/debug.c \
+	openbox/actions/decorations.c \
 	openbox/actions/directionalcyclewindows.c \
 	openbox/actions/directionaltargetwindow.c \
 	openbox/actions/execute.c \
diff --git a/openbox/action.c b/openbox/action.c
index 61c29721..2dca0dd5 100644
--- a/openbox/action.c
+++ b/openbox/action.c
@@ -432,11 +432,6 @@ ActionString actionstrings[] =
         action_desktop_dir,
         setup_action_desktop_down
     },
-    {
-        "toggledecorations",
-        action_toggle_decorations,
-        setup_client_action
-    },
     {
         "toggledockautohide",
         action_toggle_dockautohide,
@@ -890,16 +885,6 @@ void action_desktop_last(union ActionData *data)
         screen_set_desktop(screen_last_desktop, TRUE);
 }
 
-void action_toggle_decorations(union ActionData *data)
-{
-    ObClient *c = data->client.any.c;
-
-    client_action_start(data);
-    client_set_undecorated(c, !c->undecorated);
-    client_action_end(data, FALSE);
-}
-
-
 void action_directional_focus(union ActionData *data)
 {
     /* if using focus_delay, stop the timer now so that focus doesn't go moving
diff --git a/openbox/actions/all.c b/openbox/actions/all.c
index c41c5f85..7297d235 100644
--- a/openbox/actions/all.c
+++ b/openbox/actions/all.c
@@ -31,4 +31,5 @@ void action_all_startup()
     action_directionalcyclewindows_startup();
     action_directionaltargetwindow_startup();
     action_resize_startup();
+    action_decorations_startup();
 }
diff --git a/openbox/actions/all.h b/openbox/actions/all.h
index cae50318..477b5fe9 100644
--- a/openbox/actions/all.h
+++ b/openbox/actions/all.h
@@ -32,5 +32,6 @@ void action_omnipresent_startup();
 void action_directionalcyclewindows_startup();
 void action_directionaltargetwindow_startup();
 void action_resize_startup();
+void action_decorations_startup();
 
 #endif
diff --git a/openbox/actions/decorations.c b/openbox/actions/decorations.c
new file mode 100644
index 00000000..8a816ae4
--- /dev/null
+++ b/openbox/actions/decorations.c
@@ -0,0 +1,62 @@
+#include "openbox/actions.h"
+#include "openbox/client.h"
+
+typedef struct {
+    gboolean toggle;
+    gboolean on;
+} Options;
+
+static gpointer setup_func(ObParseInst *i, xmlDocPtr doc, xmlNodePtr node);
+static void     free_func(gpointer options);
+static gboolean run_func(ObActionsData *data, gpointer options);
+
+void action_decorations_startup()
+{
+    actions_register("Decorations",
+                     setup_func,
+                     free_func,
+                     run_func,
+                     NULL, NULL);
+}
+
+static gpointer setup_func(ObParseInst *i, xmlDocPtr doc, xmlNodePtr node)
+{
+    xmlNodePtr n;
+    Options *o;
+
+    o = g_new0(Options, 1);
+    o->toggle = TRUE;
+
+    if ((n = parse_find_node("decorations", node))) {
+        gchar *s = parse_string(doc, n);
+        if (g_ascii_strcasecmp(s, "toggle")) {
+            o->toggle = FALSE;
+            o->on = parse_bool(doc, n);
+        }
+        g_free(s);
+    }
+
+    return o;
+}
+
+static void free_func(gpointer options)
+{
+    Options *o = options;
+
+    g_free(o);
+}
+
+/* Always return FALSE because its not interactive */
+static gboolean run_func(ObActionsData *data, gpointer options)
+{
+    Options *o = options;
+
+    if (data->client) {
+        actions_client_move(data, TRUE);
+        if (o->toggle || o->on != !data->client->undecorated)
+            client_set_undecorated(data->client, !data->client->undecorated);
+        actions_client_move(data, FALSE);
+    }
+
+    return FALSE;
+}
-- 
2.45.2