X-Git-Url: https://git.brokenzipper.com/gitweb?a=blobdiff_plain;f=openbox%2Fmenu.c;h=fae9dca6272d54fda6dd4e69d2b9d16e833b3a16;hb=df0f00848aaa95fb2147634446b2a66fa6820865;hp=d832392ea7730963e3a4eba05164c736780b755f;hpb=d51702affa3e3895f9fca218de39690f8958b500;p=chaz%2Fopenbox

diff --git a/openbox/menu.c b/openbox/menu.c
index d832392e..fae9dca6 100644
--- a/openbox/menu.c
+++ b/openbox/menu.c
@@ -45,7 +45,9 @@ void parse_menu_full(xmlDocPtr doc, xmlNodePtr node, void *data,
             data.doc = doc;
             data.node = node;
             data.parent = menu;
-            parent = plugin_create(plugin, &data);
+
+            if (plugin_open_reopen(plugin))
+                parent = plugin_create(plugin, &data);
             g_free(plugin);
         } else
             menu = menu_new(title, id, data ? *((ObMenu**)data) : NULL);
@@ -65,7 +67,8 @@ void parse_menu_full(xmlDocPtr doc, xmlNodePtr node, void *data,
                 data.doc = doc;
                 data.node = node;
                 data.parent = menu;
-                parent = plugin_create(plugin, &data);
+                if (plugin_open_reopen(plugin))
+                    parent = plugin_create(plugin, &data);
                 g_free(plugin);
             } else {
                 parent = menu;
@@ -183,6 +186,7 @@ ObMenu *menu_new_full(char *label, char *name, ObMenu *parent,
     self->name = g_strdup(name);
     self->parent = parent;
     self->open_submenu = NULL;
+    self->over = NULL;
 
     self->entries = NULL;
     self->shown = FALSE;
@@ -296,7 +300,7 @@ void menu_show_full(ObMenu *self, int x, int y, ObClient *client)
 {
     g_assert(self != NULL);
        
-    menu_render(self);
+    self->update(self);
     
     self->client = client;
 
@@ -316,14 +320,9 @@ void menu_hide(ObMenu *self) {
         XUnmapWindow(ob_display, self->frame);
         self->shown = FALSE;
 	if (self->open_submenu)
-	    menu_hide(self->open_submenu);
+	    self->open_submenu->hide(self->open_submenu);
 	if (self->parent && self->parent->open_submenu == self) {
-            ObMenuEntry *e;
-
 	    self->parent->open_submenu = NULL;
-
-            e = menu_find_entry_by_submenu(self->parent, self);
-            self->parent->mouseover(e, FALSE);
         }
 
         if (!(self->parent && self->parent->shown)) {
@@ -331,6 +330,11 @@ void menu_hide(ObMenu *self) {
             grab_pointer(FALSE, None);
         }
         menu_visible = g_list_remove(menu_visible, self);
+        if (self->over) {
+            ((ObMenuEntry *)self->over->data)->hilite = FALSE;
+            menu_entry_render(self->over->data);
+            self->over = NULL;
+        }
     }
 }
 
@@ -397,7 +401,7 @@ void menu_entry_fire(ObMenuEntry *self, unsigned int button, unsigned int x,
         /* hide the whole thing */
         m = self->parent;
         while (m->parent) m = m->parent;
-        menu_hide(m);
+        m->hide(m);
     }
 }
 
@@ -430,7 +434,7 @@ void menu_control_show(ObMenu *self, int x, int y, ObClient *client)
         stacking_raise(MENU_AS_WINDOW(self));
 	self->shown = TRUE;
     } else if (self->shown && self->open_submenu) {
-	menu_hide(self->open_submenu);
+	self->open_submenu->hide(self->open_submenu);
     }
 }
 
@@ -440,15 +444,17 @@ void menu_control_mouseover(ObMenuEntry *self, gboolean enter)
     Rect *a;
     ObMenuEntry *e;
 
+    g_assert(self != NULL);
+    
     if (enter) {
+        /* TODO: we prolly don't need open_submenu */
 	if (self->parent->open_submenu && self->submenu 
 	    != self->parent->open_submenu)
         {
-            e = menu_find_entry_by_submenu(self->parent,
-                                           self->parent->open_submenu);
+            e = (ObMenuEntry *) self->parent->over->data;
             e->hilite = FALSE;
             menu_entry_render(e);
-	    menu_hide(self->parent->open_submenu);
+	    self->parent->open_submenu->hide(self->parent->open_submenu);
         }
 	
 	if (self->submenu && self->parent->open_submenu != self->submenu) {
@@ -462,7 +468,7 @@ void menu_control_mouseover(ObMenuEntry *self, gboolean enter)
 		ob_rr_theme->bwidth - ob_rr_theme->menu_overlap;
 
 	    /* need to get the width. is this bad?*/
-	    menu_render(self->submenu);
+	    self->parent->update(self->submenu);
 
             a = screen_physical_area_monitor(self->parent->xin_area);
 
@@ -473,113 +479,112 @@ void menu_control_mouseover(ObMenuEntry *self, gboolean enter)
 	    menu_show_full(self->submenu, x,
 			   self->parent->location.y + self->y,
                            self->parent->client);
-	} 
-    }
-
-    if (enter || !self->submenu ||
-        menu_find_entry_by_submenu(self->parent,
-                                   self->parent->open_submenu) != self)
-        self->hilite = enter;
-
+	}
+        self->hilite = TRUE;
+        self->parent->over = g_list_find(self->parent->entries, self);
+        
+    } else
+        self->hilite = FALSE;
+    
     menu_entry_render(self);
 }
 
-ObMenuEntry *menu_control_keyboard_nav(ObMenuEntry *over, ObKey key)
+void menu_control_keyboard_nav(unsigned int key)
 {
-    GList *it = NULL;
-        
-    switch (key) {
+    static ObMenu *current_menu = NULL;
+    ObMenuEntry *e = NULL;
+
+    ObKey obkey = OB_NUM_KEYS;
+
+    /* hrmm. could be fixed */
+    if (key == ob_keycode(OB_KEY_DOWN))
+        obkey = OB_KEY_DOWN;
+    else if (key == ob_keycode(OB_KEY_UP))
+        obkey = OB_KEY_UP;
+    else if (key == ob_keycode(OB_KEY_RIGHT)) /* fuck */
+        obkey = OB_KEY_RIGHT;
+    else if (key == ob_keycode(OB_KEY_LEFT)) /* users */
+        obkey = OB_KEY_LEFT;
+    else if (key == ob_keycode(OB_KEY_RETURN))
+        obkey = OB_KEY_RETURN;
+
+    
+    if (current_menu == NULL)
+        current_menu = menu_visible->data;
+    
+    switch (obkey) {
     case OB_KEY_DOWN: {
-        if (over != NULL) {
-            over->parent->mouseover(over, FALSE);
-                
-            it = over->parent->entries;
-            while (it != NULL && it->data != over)
-                it = it->next;
-        }
-            
-        if (it && it->next)
-            over = (ObMenuEntry *)it->next->data;
-        else if (over == NULL) {
-            if (menu_visible && ((ObMenu *)menu_visible->data)->entries)
-                over = (ObMenuEntry *)
-                    (((ObMenu *)menu_visible->data)->entries)->data;
-            else
-                over = NULL;
-        } else {
-            over = (over->parent->entries != NULL ?
-                    over->parent->entries->data : NULL);
+        if (current_menu->over) {
+            current_menu->mouseover(current_menu->over->data, FALSE);
+            current_menu->over = (current_menu->over->next != NULL ?
+                          current_menu->over->next :
+                          current_menu->entries);
         }
+        else
+            current_menu->over = current_menu->entries;
 
-        if (over)
-            over->parent->mouseover(over, TRUE);
+        if (current_menu->over)
+            current_menu->mouseover(current_menu->over->data, TRUE);
         
         break;
     }
     case OB_KEY_UP: {
-        if (over != NULL) {
-            over->parent->mouseover(over, FALSE);
-                
-            it = g_list_last(over->parent->entries);
-            while (it != NULL && it->data != over)
-                it = it->prev;
-        } 
-            
-        if (it && it->prev)
-            over = (ObMenuEntry *)it->prev->data;
-        else if (over == NULL) {
-            it = g_list_last(menu_visible);
-            if (it != NULL) {
-                it = g_list_last(((ObMenu *)it->data)->entries);
-                over = (ObMenuEntry *)(it != NULL ? it->data : NULL);
-            }
+        if (current_menu->over) {
+            current_menu->mouseover(current_menu->over->data, FALSE);
+            current_menu->over = (current_menu->over->prev != NULL ?
+                          current_menu->over->prev :
+                g_list_last(current_menu->entries));
         } else
-            over = (over->parent->entries != NULL ?
-                    g_list_last(over->parent->entries)->data :
-                    NULL);
+            current_menu->over = g_list_last(current_menu->entries);
 
-        over->parent->mouseover(over, TRUE);
+        if (current_menu->over)
+            current_menu->mouseover(current_menu->over->data, TRUE);
+        
         break;
     }
     case OB_KEY_RIGHT: {
-        if (over == NULL)
-            return over;
-
-        if (over->submenu) {
-            over->parent->mouseover(over, FALSE);
-
-            if (over->submenu->entries)
-                over = over->submenu->entries->data;
-
-            over->parent->mouseover(over, TRUE);
-        }
-        else {
-            over->parent->mouseover(over, FALSE);
-
-            /* zero is enter */
-            menu_entry_fire(over, 0, 0, 0);
+        if (current_menu->over == NULL)
+            return;
+        e = (ObMenuEntry *)current_menu->over->data;
+        if (e->submenu) {
+            current_menu->mouseover(e, TRUE);
+            current_menu = e->submenu;
+            current_menu->over = current_menu->entries;
+            if (current_menu->over)
+                current_menu->mouseover(current_menu->over->data, TRUE);
         }
         break;
     }
+
+    case OB_KEY_RETURN: {
+        if (current_menu->over == NULL)
+            return;
+        e = (ObMenuEntry *)current_menu->over->data;
+
+        current_menu->mouseover(e, FALSE);
+        current_menu->over = NULL;
+        /* zero is enter */
+        menu_entry_fire(e, 0, 0, 0);
+    }
+        
     case OB_KEY_LEFT: {
-        if (over != NULL) {
-            over->parent->mouseover(over, FALSE);
-            menu_hide(over->parent);
-        } else {
-            it  = g_list_last(menu_visible);
-            if (it) {
-                menu_hide((ObMenu *)it->data);
-            }
+        if (current_menu->over != NULL) {
+            current_menu->mouseover(current_menu->over->data, FALSE);
+            current_menu->over = NULL;
         }
         
-        over = NULL;
+        current_menu->hide(current_menu);
+
+        if (current_menu->parent)
+            current_menu = current_menu->parent;
+        
         break;
     }
     default:
-        g_error("Unknown key");
+        ((ObMenu *)menu_visible->data)->hide(menu_visible->data);
+        current_menu = NULL;
     }
-
-    return over;
+    return;
 }
 
 void menu_noop()