ayatana-commits team mailing list archive

[Cody Russell <crussell@xxxxxxxxxxxxx>]

Cody Russell has proposed merging lp:~bratsche/appmenu-gtk/invisible-separators into lp:appmenu-gtk.

Requested reviews:
  Canonical Desktop Experience Team (canonical-dx-team)
Related bugs:
  #598244 Unexpected separators in menus
  https://bugs.launchpad.net/bugs/598244


I think this is worth getting in and backporting into A2, even though I'm a little confused by it.  This actually seems like it may be closely related to the problem we're having with dynamic menus.
-- 
https://code.launchpad.net/~bratsche/appmenu-gtk/invisible-separators/+merge/28455
Your team ayatana-commits is subscribed to branch lp:appmenu-gtk.

=== modified file 'src/bridge.c'
--- src/bridge.c	2010-06-24 16:31:17 +0000
+++ src/bridge.c	2010-06-24 20:02:25 +0000
@@ -506,6 +506,11 @@
                 }
             }
 
+          if (!gtk_widget_get_visible (widget))
+            {
+              skip = TRUE;
+            }
+
           if (!skip)
             {
               recurse->stack[recurse->count] = construct_dbusmenu_for_widget (widget);