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);
_______________________________________________
Mailing list: https://launchpad.net/~ayatana-commits
Post to : [email protected]
Unsubscribe : https://launchpad.net/~ayatana-commits
More help : https://help.launchpad.net/ListHelp
