ayatana-commits team mailing list archive

[Ted Gould <ted@xxxxxxxx>]

Ted Gould has proposed merging lp:~ted/indicator-messages/extra-separator into lp:indicator-messages.

    Requested reviews:
    Indicator Applet Developers (indicator-applet-developers)


An extra separator with black listed launchers being at the end.
-- 
https://code.launchpad.net/~ted/indicator-messages/extra-separator/+merge/12868
Your team ayatana-commits is subscribed to branch lp:indicator-messages.

=== modified file 'src/messages-service.c'
--- src/messages-service.c	2009-09-29 21:04:25 +0000
+++ src/messages-service.c	2009-10-05 13:30:22 +0000
@@ -1147,7 +1147,6 @@
 		/* Add it to the menu */
 		dbusmenu_menuitem_child_append(root_menuitem, DBUSMENU_MENUITEM(ll->menuitem));
 		dbusmenu_menuitem_child_append(root_menuitem, DBUSMENU_MENUITEM(ll->separator));
-		resort_menu(root_menuitem);
 
 		/* If we're in the black list or we've gotten eclipsed
 		   by something else, hide the item and the separator. */
@@ -1157,6 +1156,7 @@
 			dbusmenu_menuitem_property_set(ll->separator, DBUSMENU_MENUITEM_PROP_VISIBLE, "false");
 		}
 
+		resort_menu(root_menuitem);
 		check_hidden();
 	} else {
 		/* If so add ourselves */