all repos — openbox @ d915480371427b6813e7a5140caa4f15c30ee5c9

openbox fork - make it a bit more like ryudo 

oops hehe. a render call in the loop for debugging was what made it slow. anyhow make it even faster by only calling render once too. yay snappiness.
Dana Jansens danakj@orodu.net
commit

d915480371427b6813e7a5140caa4f15c30ee5c9

parent

3905872982478e4e0ae05f4ce114a8025a2389ee

1 files changed, 11 insertions(+), 9 deletions(-)

jump to
M openbox/menuframe.c → openbox/menuframe.c 
@@ -824,13 +824,18 @@ self->entries = g_list_delete_link(self->entries, fit);
         fit = n;
     }
 
-    menu_frame_render(self);
+    /* * make the menu fit on the screen */
 
-    /* make the menu fit on the screen. at most we call render twice, at least
-       not like n times or sometime */
+    /* calculate the height of the menu */
+    h = 0;
+    for (fit = self->entries; fit; fit = g_list_next(fit))
+        h += menu_entry_frame_get_height(fit->data,
+                                         fit == self->entries,
+                                         g_list_next(fit) == NULL);
+    /* add the border at the top and bottom */
+    h += ob_rr_theme->mbwidth * 2;
 
     a = screen_physical_area_monitor(self->monitor);
-    h = self->area.height;
 
     if (h > a->height) {
         GList *flast, *tmp;

@@ -853,8 +858,6 @@ tmp = flast;
             flast = g_list_previous(flast);
             menu_entry_frame_free(tmp->data);
             self->entries = g_list_delete_link(self->entries, tmp);
-
-            menu_frame_render(self);
 
             /* only the first one that we see is the last entry in the menu */
             last_entry = FALSE;

@@ -881,10 +884,9 @@ 
             /* add our More... entry to the frame */
             self->entries = g_list_append(self->entries, more_frame);
         }
+    }
 
-        /* render again */
-        menu_frame_render(self);
-    }
+    menu_frame_render(self);
 }
 
 static gboolean menu_frame_is_visible(ObMenuFrame *self)