ayatana-commits team mailing list archive

[Michael Terry <michael.terry@xxxxxxxxxxxxx>]

Michael Terry has proposed merging lp:~mterry/appmenu-gtk/toggle-state-is-int into lp:appmenu-gtk.

Requested reviews:
  Canonical Desktop Experience Team (canonical-dx-team)
Related bugs:
  #682075 keyboard indicator has dot beside wrong layout
  https://bugs.launchpad.net/bugs/682075

For more details, see:
https://code.launchpad.net/~mterry/appmenu-gtk/toggle-state-is-int/+merge/47659

I believe I proposed this before, but the proposal seems gone.
-- 
https://code.launchpad.net/~mterry/appmenu-gtk/toggle-state-is-int/+merge/47659
Your team ayatana-commits is subscribed to branch lp:appmenu-gtk.

=== modified file 'src/bridge.c'
--- src/bridge.c	2011-01-21 16:34:43 +0000
+++ src/bridge.c	2011-01-27 14:14:58 +0000
@@ -692,9 +692,9 @@
     }
   else if (pspec->name == g_intern_static_string ("active"))
     {
-      dbusmenu_menuitem_property_set_bool (mi,
-                                           DBUSMENU_MENUITEM_PROP_TOGGLE_STATE,
-                                           gtk_toggle_action_get_active (GTK_TOGGLE_ACTION (action)));
+      dbusmenu_menuitem_property_set_int (mi,
+                                          DBUSMENU_MENUITEM_PROP_TOGGLE_STATE,
+                                          gtk_toggle_action_get_active (GTK_TOGGLE_ACTION (action)) ? DBUSMENU_MENUITEM_TOGGLE_STATE_CHECKED : DBUSMENU_MENUITEM_TOGGLE_STATE_UNCHECKED);
     }
   else if (pspec->name == g_intern_static_string ("label"))
     {