X-Git-Url: https://git.dogcows.com/gitweb?a=blobdiff_plain;f=openbox%2Faction.c;h=33950daa26d4b5cb39c8448f4d3f359337d0c4d2;hb=0e28a07e3d6677aa6af9ad97fbc55f8101f3fdf2;hp=cc934206ce58935654a6c77a09868b716070d310;hpb=d498522a560529ef315dd30fc4bc14c7a6214b30;p=chaz%2Fopenbox

diff --git a/openbox/action.c b/openbox/action.c
index cc934206..33950daa 100644
--- a/openbox/action.c
+++ b/openbox/action.c
@@ -1,13 +1,13 @@
 #include "client.h"
 #include "focus.h"
+#include "menu.h"
 #include "stacking.h"
 #include "frame.h"
+#include "framerender.h"
 #include "screen.h"
 #include "action.h"
 #include "dispatch.h"
 #include "openbox.h"
-#include "engine.h"
-#include "render/render.h"
 
 #include <glib.h>
 
@@ -16,10 +16,6 @@ Action *action_new(void (*func)(union ActionData *data))
     Action *a = g_new0(Action, 1);
     a->func = func;
 
-    /* deal with pointers */
-    if (func == action_execute)
-        a->data.execute.path = NULL;
-
     return a;
 }
 
@@ -30,6 +26,8 @@ void action_free(Action *a)
     /* deal with pointers */
     if (a->func == action_execute || a->func == action_restart)
         g_free(a->data.execute.path);
+    else if (a->func == action_showmenu)
+        g_free(a->data.showmenu.name);
 
     g_free(a);
 }
@@ -645,16 +643,14 @@ static void popup_coords(char *format, int a, int b, gboolean hide)
     if (hide)
         XUnmapWindow(ob_display, coords);
     else {
-        Rect area = { 10, 10, 1, 1 };
         Size s;
         char *text;
 
         text = g_strdup_printf(format, a, b);
-        engine_size_label(text, TRUE, TRUE, &s);
-        area.width = s.width; area.height = s.height;
+        framerender_size_popup_label(text, &s);
         XMoveResizeWindow(ob_display, coords,
-                          area.x, area.y, area.width, area.height);
-        engine_render_label(coords, &area, text, TRUE, TRUE);
+                          10, 10, s.width, s.height);
+        framerender_popup_label(coords, &s, text);
         g_free(text);
 
         XMapWindow(ob_display, coords);
@@ -711,59 +707,48 @@ void action_exit(union ActionData *data)
 
 void action_showmenu(union ActionData *data)
 {
-    g_message(__FUNCTION__);
+    menu_show(data->showmenu.name, data->showmenu.x, data->showmenu.y,
+              data->showmenu.c);
 }
 
-void action_cycle_windows(union ActionData *data)
+static void popup_cycle(Client *c, gboolean hide)
 {
-    static Client *first = NULL;
-    static Client *t = NULL;
-    static GList *order = NULL;
-    GList *it, *start, *list;
+    XSetWindowAttributes attrib;
+    static Window coords = None;
 
-    if (data->cycle.cancel) {
-        if (first) client_focus(first);
-        goto done_cycle;
+    if (coords == None) {
+        attrib.override_redirect = TRUE;
+        coords = XCreateWindow(ob_display, ob_root,
+                               0, 0, 1, 1, 0, render_depth, InputOutput,
+                               render_visual, CWOverrideRedirect, &attrib);
+        g_assert(coords != None);
     }
-    if (!first) first = focus_client;
 
-    if (data->cycle.linear)
-        list = client_list;
+    if (hide)
+        XUnmapWindow(ob_display, coords);
     else {
-        if (!order) order = g_list_copy(focus_order[screen_desktop]);
-        list = order;
-    }
-    start = it = g_list_find(list, data->cycle.c);
-    if (!start) goto done_cycle;
-
-    if (!data->cycle.final) {
-        t = NULL;
-        if (!start) /* switched desktops or something? */
-            goto done_cycle;
-
-        do {
-            if (data->cycle.forward) {
-                it = it->next;
-                if (it == NULL) it = list;
-            } else {
-                it = it->prev;
-                if (it == NULL) it = g_list_last(list);
-            }
-            if (client_focus(it->data)) {
-                t = it->data;
-                focus_ignore_in++;
-                break;
-            }
-        } while (it != start);
-    } else {
-        if (t) stacking_raise(t);
-        goto done_cycle;
+        Rect *a;
+        Size s;
+
+        a = screen_area(c->desktop);
+
+        framerender_size_popup_label(c->title, &s);
+        XMoveResizeWindow(ob_display, coords,
+                          a->x + (a->width - s.width) / 2,
+                          a->y + (a->height - s.height) / 2,
+                          s.width, s.height);
+        framerender_popup_label(coords, &s, c->title);
+
+        XMapWindow(ob_display, coords);
     }
-    return;
+}
 
-    done_cycle:
-        first = NULL;
-        g_list_free(order);
-        order = NULL;
+void action_cycle_windows(union ActionData *data)
+{
+    Client *c;
+    
+    c = focus_cycle(data->cycle.forward, data->cycle.linear, data->cycle.final,
+                    data->cycle.cancel);
+    popup_cycle(c, !c || data->cycle.final || data->cycle.cancel);
 }