← Back to team overview

ayatana-commits team mailing list archive

[Merge] lp:~bratsche/appmenu-gtk/menubar-visibility into lp:appmenu-gtk

 

Cody Russell has proposed merging lp:~bratsche/appmenu-gtk/menubar-visibility into lp:appmenu-gtk.

Requested reviews:
  Canonical Desktop Experience Team (canonical-dx-team)
Related bugs:
  #608967 Menubars appear/disappear randomly
  https://bugs.launchpad.net/bugs/608967

-- 
https://code.launchpad.net/~bratsche/appmenu-gtk/menubar-visibility/+merge/30731
Your team ayatana-commits is subscribed to branch lp:appmenu-gtk.
=== modified file 'src/bridge.c'
--- src/bridge.c	2010-07-22 13:15:29 +0000
+++ src/bridge.c	2010-07-23 00:12:43 +0000
@@ -338,12 +338,12 @@
 {
   if (strlen (new) > 0 && strlen (prev) == 0)
     {
-      if (g_strcmp0 (name, APP_MENU_DBUS_NAME) == 0)
+      if (g_strcmp0 (name, APP_MENU_DBUS_NAME) == 0 && (prev == NULL || strlen (prev) == 0))
         {
           app_menu_bridge_proxy_appeared (bridge);
         }
     }
-  else if (strlen (new) == 0 && strlen (prev) > 0)
+  else if (g_strcmp0 (name, APP_MENU_DBUS_NAME) == 0 && strlen (new) == 0 && strlen (prev) > 0)
     {
       app_menu_bridge_proxy_vanished (bridge);
     }


Follow ups