Merge lp:~dbusmenu-team/libdbusmenu/ubuntu into lp:~ubuntu-desktop/libdbusmenu/ubuntu

[Ted Gould]

0.4.2

[lp:~dbusmenu-team/libdbusmenu/ubuntu]

138. By Ken VanDine on 2011-04-07

releasing version 0.4.2-0ubuntu1

=== modified file 'ChangeLog'
--- ChangeLog	2011-03-31 18:48:12 +0000
+++ ChangeLog	2011-04-07 15:38:25 +0000
@@ -1,5 +1,77 @@
 # Generated by Makefile. Do not edit.
 
[phone]  Ted Gould  <[e-mail]>
+
+	0.4.2
+
[phone]  Ted Gould  <[e-mail]>
+
+	Build menus based on teh child-display property
+
[phone]  Ted Gould  <[e-mail]>
+
+	Make check to send about-to-show look for properties as well.
+
[phone]  Ted Gould  <[e-mail]>
+
+	Attching a bug number
+
[phone]  Ted Gould  <[e-mail]>
+
+	Making the debug message better
+
[phone]  Ted Gould  <[e-mail]>
+
+	We don't need the parent, cut-and-paste confusion here.
+
[phone]  Ted Gould  <[e-mail]>
+
+	Handle the case of the variant being NULL
+
[phone]  Ted Gould  <[e-mail]>
+
+	Now that we're using the child-display variable more, let's grab it in the first pass.
+
[phone]  Ted Gould  <[e-mail]>
+
+	Then menu should get created when processing the property, so let's warn and exit here if it's not available
+
[phone]  Ted Gould  <[e-mail]>
+
+	Passing in the client so that I can get the menuitem in all cases
+
[phone]  Ted Gould  <[e-mail]>
+
+	Setting up the shell to handle the children display prop
+
[phone]  Ted Gould  <[e-mail]>
+
+	Add a handler for the open/close events and pass them up to GTK
+
[phone]  Michael Terry  <[e-mail]>
+
+	respond to opened and closed events for the benefit of any watching application
+
[phone]  Ted Gould  <[e-mail]>
+
+	Keep refs to the client and makes sure to disconnect all handlers.
+
[phone]  Michael Terry  <[e-mail]>
+
+	fix typo
+
[phone]  Michael Terry  <[e-mail]>
+
+	be a little more careful with asynchronous callbacks
+
[phone]  Ted Gould  <[e-mail]>
+
+	Don't duplicate flags
+
[phone]  Ted Gould  <[e-mail]>
+
+	Don't need SCANNERFLAGS and INTROSPECTION_SCANNER_ARGS both so they dup'd the parameters.
+
 2011-03-31  Ted Gould  <[e-mail]>
 
 	0.4.1

=== modified file 'configure'
--- configure	2011-03-31 18:50:13 +0000
+++ configure	2011-04-07 15:38:25 +0000
@@ -1,6 +1,6 @@
 #! /bin/sh
 # Guess values for system-dependent variables and create Makefiles.
-# Generated by GNU Autoconf 2.67 for libdbusmenu 0.4.1.
+# Generated by GNU Autoconf 2.67 for libdbusmenu 0.4.2.
 #
 # Report bugs to <[e-mail]>.
 #
@@ -703,8 +703,8 @@
 # Identity of this package.
 PACKAGE_NAME='libdbusmenu'
 PACKAGE_TARNAME='libdbusmenu'
-PACKAGE_VERSION='0.4.1'
-PACKAGE_STRING='libdbusmenu 0.4.1'
+PACKAGE_VERSION='0.4.2'
+PACKAGE_STRING='libdbusmenu 0.4.2'
 PACKAGE_BUGREPORT='[e-mail]'
 PACKAGE_URL=''
 
@@ -1541,7 +1541,7 @@
   # Omit some internal or obsolete options to make the list less imposing.
   # This message is too long to be a string in the A/UX 3.1 sh.
   cat <<_ACEOF
-\`configure' configures libdbusmenu 0.4.1 to adapt to many kinds of systems.
+\`configure' configures libdbusmenu 0.4.2 to adapt to many kinds of systems.
 
 Usage: $0 [OPTION]... [VAR=VALUE]...
 
@@ -1611,7 +1611,7 @@
 
 if test -n "$ac_init_help"; then
   case $ac_init_help in
-     short | recursive ) echo "Configuration of libdbusmenu 0.4.1:";;
+     short | recursive ) echo "Configuration of libdbusmenu 0.4.2:";;
    esac
   cat <<\_ACEOF
 
@@ -1751,7 +1751,7 @@
 test -n "$ac_init_help" && exit $ac_status
 if $ac_init_version; then
   cat <<\_ACEOF
-libdbusmenu configure 0.4.1
+libdbusmenu configure 0.4.2
 generated by GNU Autoconf 2.67
 
 Copyright (C) 2010 Free Software Foundation, Inc.
@@ -2122,7 +2122,7 @@
 This file contains any messages produced by compilers while
 running configure, to aid debugging if configure makes a mistake.
 
-It was created by libdbusmenu $as_me 0.4.1, which was
+It was created by libdbusmenu $as_me 0.4.2, which was
 generated by GNU Autoconf 2.67.  Invocation command line was
 
   $ $0 $@
@@ -2943,7 +2943,7 @@
 
 # Define the identity of the package.
  PACKAGE=libdbusmenu
- VERSION=0.4.1
+ VERSION=0.4.2
 
 
 # Some tools Automake needs.
@@ -13294,7 +13294,7 @@
 ###########################
 
 LIBDBUSMENU_CURRENT=3
-LIBDBUSMENU_REVISION=12
+LIBDBUSMENU_REVISION=13
 LIBDBUSMENU_AGE=0
 
 
@@ -14616,7 +14616,7 @@
 # report actual input values of CONFIG_FILES etc. instead of their
 # values after options handling.
 ac_log="
-This file was extended by libdbusmenu $as_me 0.4.1, which was
+This file was extended by libdbusmenu $as_me 0.4.2, which was
 generated by GNU Autoconf 2.67.  Invocation command line was
 
   CONFIG_FILES    = $CONFIG_FILES
@@ -14682,7 +14682,7 @@
 cat >>$CONFIG_STATUS <<_ACEOF || ac_write_fail=1
 ac_cs_config="`$as_echo "$ac_configure_args" | sed 's/^ //; s/[\\""\`\$]/\\\\&/g'`"
 ac_cs_version="\\
-libdbusmenu config.status 0.4.1
+libdbusmenu config.status 0.4.2
 configured by $0, generated by GNU Autoconf 2.67,
   with options \\"\$ac_cs_config\\"
 

=== modified file 'configure.ac'
--- configure.ac	2011-03-31 18:50:13 +0000
+++ configure.ac	2011-04-07 15:38:25 +0000
@@ -1,11 +1,11 @@
 
-AC_INIT(libdbusmenu, 0.4.1, [e-mail])
+AC_INIT(libdbusmenu, 0.4.2, [e-mail])
 AC_COPYRIGHT([Copyright 2009,2010 Canonical])
 
 AC_PREREQ(2.62)
 
 AM_CONFIG_HEADER(config.h)
-AM_INIT_AUTOMAKE(libdbusmenu, 0.4.1, [-Wno-portability])
+AM_INIT_AUTOMAKE(libdbusmenu, 0.4.2, [-Wno-portability])
 
 AM_MAINTAINER_MODE
 
@@ -134,7 +134,7 @@
 ###########################
 
 LIBDBUSMENU_CURRENT=3
-LIBDBUSMENU_REVISION=12
+LIBDBUSMENU_REVISION=13
 LIBDBUSMENU_AGE=0
 
 AC_SUBST(LIBDBUSMENU_CURRENT)

=== modified file 'debian/changelog'
--- debian/changelog	2011-03-31 19:33:18 +0000
+++ debian/changelog	2011-04-07 15:38:25 +0000
@@ -1,3 +1,17 @@
+libdbusmenu (0.4.2-0ubuntu1~ppa1) natty; urgency=low
+
+  * New upstream release.
+    ∘ Fix GIR scan to not include duplicate namespaces (LP: #750575)
+    ∘ Keep refs to the client and makes sure to disconnect all handlers.
+      (LP: #749609)
+    ∘ Add GTK side handlers for the open/close events to ensure GTK apps
+      can use those events still (LP: #750588)
+    ∘ Create submenus for menuitems that don't have children but still
+      have told us they need them with the child-display property.
+      (LP: #619811)
+
+ -- Ted Gould <[e-mail]>  Thu, 07 Apr 2011 10:32:51 -0500
+
 libdbusmenu (0.4.1-0ubuntu1) natty; urgency=low
 
   * New upstream release.

=== modified file 'docs/libdbusmenu-glib/reference/html/ch01.html'
--- docs/libdbusmenu-glib/reference/html/ch01.html	2011-03-31 18:48:12 +0000
+++ docs/libdbusmenu-glib/reference/html/ch01.html	2011-04-07 15:38:25 +0000
@@ -21,7 +21,7 @@
 </tr></table>
 <div class="chapter">
 <div class="titlepage"><div><div><h2 class="title">
-<a name="id471000"></a>API</h2></div></div></div>
+<a name="id552681"></a>API</h2></div></div></div>
 <div class="toc"><dl>
 <dt>
 <span class="refentrytitle"><a href="libdbusmenu-glib-DbusmenuServer.html">DbusmenuServer</a></span><span class="refpurpose"> — The server signals changed and

=== modified file 'docs/libdbusmenu-glib/reference/version.xml'
--- docs/libdbusmenu-glib/reference/version.xml	2011-03-31 18:50:13 +0000
+++ docs/libdbusmenu-glib/reference/version.xml	2011-04-07 15:38:25 +0000
@@ -1,1 +1,1 @@
-0.4.1
+0.4.2

=== modified file 'docs/libdbusmenu-gtk/reference/html/ch01.html'
--- docs/libdbusmenu-gtk/reference/html/ch01.html	2011-03-31 18:48:12 +0000
+++ docs/libdbusmenu-gtk/reference/html/ch01.html	2011-04-07 15:38:25 +0000
@@ -21,7 +21,7 @@
 </tr></table>
 <div class="chapter">
 <div class="titlepage"><div><div><h2 class="title">
-<a name="id520868"></a>API</h2></div></div></div>
+<a name="id463422"></a>API</h2></div></div></div>
 <div class="toc"><dl>
 <dt>
 <span class="refentrytitle"><a href="libdbusmenu-gtk-DbusmenuGtkMenu.html">DbusmenuGtkMenu</a></span><span class="refpurpose"> — A GTK Menu Object that syncronizes over DBus</span>

=== modified file 'docs/libdbusmenu-gtk/reference/version.xml'
--- docs/libdbusmenu-gtk/reference/version.xml	2011-03-31 18:50:13 +0000
+++ docs/libdbusmenu-gtk/reference/version.xml	2011-04-07 15:38:25 +0000
@@ -1,1 +1,1 @@
-0.4.1
+0.4.2

=== modified file 'libdbusmenu-glib/client.c'
--- libdbusmenu-glib/client.c	2011-03-31 18:47:46 +0000
+++ libdbusmenu-glib/client.c	2011-04-07 15:38:25 +0000
@@ -331,6 +331,8 @@
 	return;
 }
 
+#define LAYOUT_PROPS_COUNT  5
+
 static void
 dbusmenu_client_init (DbusmenuClient *self)
 {
@@ -351,13 +353,14 @@
 
 	priv->layoutcall = NULL;
 
-	gchar * layout_props[5];
+	gchar * layout_props[LAYOUT_PROPS_COUNT + 1];
 	layout_props[0] = DBUSMENU_MENUITEM_PROP_TYPE;
 	layout_props[1] = DBUSMENU_MENUITEM_PROP_LABEL;
 	layout_props[2] = DBUSMENU_MENUITEM_PROP_VISIBLE;
 	layout_props[3] = DBUSMENU_MENUITEM_PROP_ENABLED;
-	layout_props[4] = NULL;
-	priv->layout_props = g_variant_new_strv((const gchar * const *)layout_props, 4);
+	layout_props[4] = DBUSMENU_MENUITEM_PROP_CHILD_DISPLAY;
+	layout_props[LAYOUT_PROPS_COUNT] = NULL;
+	priv->layout_props = g_variant_new_strv((const gchar * const *)layout_props, LAYOUT_PROPS_COUNT);
 	g_variant_ref_sink(priv->layout_props);
 
 	priv->current_revision = 0;
@@ -441,6 +444,9 @@
 		priv->menuproxy_cancel = NULL;
 	}
 	if (priv->menuproxy != NULL) {
+		g_signal_handlers_disconnect_matched(priv->menuproxy,
+		                                     G_SIGNAL_MATCH_DATA,
+		                                     0, 0, NULL, NULL, object);
 		g_object_unref(G_OBJECT(priv->menuproxy));
 		priv->menuproxy = NULL;
 	}
@@ -1494,6 +1500,7 @@
 	g_variant_unref(edata->variant);
 	g_free(edata->event);
 	g_object_unref(edata->menuitem);
+	g_object_unref(edata->client);
 	g_free(edata);
 
 	if (G_UNLIKELY(error != NULL)) {
@@ -1528,6 +1535,7 @@
 
 	event_data_t * edata = g_new0(event_data_t, 1);
 	edata->client = client;
+	g_object_ref(client);
 	edata->menuitem = mi;
 	g_object_ref(edata->menuitem);
 	edata->event = g_strdup(name);

=== modified file 'libdbusmenu-glib/menuitem.c'
--- libdbusmenu-glib/menuitem.c	2011-03-16 19:02:13 +0000
+++ libdbusmenu-glib/menuitem.c	2011-04-07 15:38:25 +0000
@@ -446,7 +446,7 @@
 {
 	g_return_if_fail(DBUSMENU_IS_MENUITEM(mi));
 
-	if (dbusmenu_menuitem_get_children(mi) == NULL) {
+	if (dbusmenu_menuitem_get_children(mi) == NULL && g_strcmp0(DBUSMENU_MENUITEM_CHILD_DISPLAY_SUBMENU, dbusmenu_menuitem_property_get(mi, DBUSMENU_MENUITEM_PROP_CHILD_DISPLAY)) != 0) {
 		g_warning("About to Show called on an item wihtout submenus.  We're ignoring it.");
 	} else {
 		gboolean dummy;

=== modified file 'libdbusmenu-gtk/Makefile.am'
--- libdbusmenu-gtk/Makefile.am	2011-02-24 18:40:30 +0000
+++ libdbusmenu-gtk/Makefile.am	2011-04-07 15:38:25 +0000
@@ -103,7 +103,6 @@
 DbusmenuGtk_0_4_gir_LIBS = libdbusmenu-gtk$(VER).la
 DbusmenuGtk_0_4_gir_FILES = $(addprefix $(srcdir)/, $(introspection_sources))
 DbusmenuGtk_0_4_gir_NAMESPACE = DbusmenuGtk$(VER)
-DbusmenuGtk_0_4_gir_SCANNERFLAGS = $(INTROSPECTION_SCANNER_ARGS)
 DbusmenuGtk_0_4_gir_EXPORT_PACKAGES = dbusmenu-gtk$(VER)-0.4
 
 # We duplicate these for the same reason as libdbusmenu_gtk3includedir above
@@ -112,7 +111,6 @@
 DbusmenuGtk3_0_4_gir_LIBS = $(DbusmenuGtk_0_4_gir_LIBS)
 DbusmenuGtk3_0_4_gir_FILES = $(DbusmenuGtk_0_4_gir_FILES)
 DbusmenuGtk3_0_4_gir_NAMESPACE = $(DbusmenuGtk_0_4_gir_NAMESPACE)
-DbusmenuGtk3_0_4_gir_SCANNERFLAGS = $(DbusmenuGtk_0_4_gir_SCANNERFLAGS)
 DbusmenuGtk3_0_4_gir_EXPORT_PACKAGES = $(DbusmenuGtk_0_4_gir_EXPORT_PACKAGES)
 
 INTROSPECTION_GIRS += DbusmenuGtk$(VER)-0.4.gir

=== modified file 'libdbusmenu-gtk/Makefile.in'
--- libdbusmenu-gtk/Makefile.in	2011-03-10 17:05:16 +0000
+++ libdbusmenu-gtk/Makefile.in	2011-04-07 15:38:25 +0000
@@ -401,7 +401,6 @@
 @HAVE_INTROSPECTION_TRUE@DbusmenuGtk_0_4_gir_LIBS = libdbusmenu-gtk$(VER).la
 @HAVE_INTROSPECTION_TRUE@DbusmenuGtk_0_4_gir_FILES = $(addprefix $(srcdir)/, $(introspection_sources))
 @HAVE_INTROSPECTION_TRUE@DbusmenuGtk_0_4_gir_NAMESPACE = DbusmenuGtk$(VER)
-@HAVE_INTROSPECTION_TRUE@DbusmenuGtk_0_4_gir_SCANNERFLAGS = $(INTROSPECTION_SCANNER_ARGS)
 @HAVE_INTROSPECTION_TRUE@DbusmenuGtk_0_4_gir_EXPORT_PACKAGES = dbusmenu-gtk$(VER)-0.4
 
 # We duplicate these for the same reason as libdbusmenu_gtk3includedir above
@@ -410,7 +409,6 @@
 @HAVE_INTROSPECTION_TRUE@DbusmenuGtk3_0_4_gir_LIBS = $(DbusmenuGtk_0_4_gir_LIBS)
 @HAVE_INTROSPECTION_TRUE@DbusmenuGtk3_0_4_gir_FILES = $(DbusmenuGtk_0_4_gir_FILES)
 @HAVE_INTROSPECTION_TRUE@DbusmenuGtk3_0_4_gir_NAMESPACE = $(DbusmenuGtk_0_4_gir_NAMESPACE)
-@HAVE_INTROSPECTION_TRUE@DbusmenuGtk3_0_4_gir_SCANNERFLAGS = $(DbusmenuGtk_0_4_gir_SCANNERFLAGS)
 @HAVE_INTROSPECTION_TRUE@DbusmenuGtk3_0_4_gir_EXPORT_PACKAGES = $(DbusmenuGtk_0_4_gir_EXPORT_PACKAGES)
 @HAVE_INTROSPECTION_TRUE@girdir = $(datadir)/gir-1.0
 @HAVE_INTROSPECTION_TRUE@gir_DATA = $(INTROSPECTION_GIRS)

=== modified file 'libdbusmenu-gtk/client.c'
--- libdbusmenu-gtk/client.c	2011-03-16 15:27:00 +0000
+++ libdbusmenu-gtk/client.c	2011-04-07 15:38:25 +0000
@@ -551,11 +551,43 @@
 	return;
 }
 
+/* Submenu processing */
+static void
+process_submenu (DbusmenuMenuitem * mi, GtkMenuItem * gmi, GVariant * variant, DbusmenuGtkClient * gtkclient)
+{
+	const gchar * submenu = NULL;
+	if (variant != NULL) {
+		submenu = g_variant_get_string(variant, NULL);
+	}
+
+	if (g_strcmp0(submenu, DBUSMENU_MENUITEM_CHILD_DISPLAY_SUBMENU) != 0) {
+		/* This is the only case we're really supporting right now,
+		   so if it's not this, we want to clean up. */
+		/* We're just going to warn for now. */
+		gpointer pmenu = g_object_get_data(G_OBJECT(mi), data_menu);
+		if (pmenu != NULL) {
+			g_warning("The child-display variable is set to '%s' but there's a menu, odd?", submenu);
+		}
+	} else {
+		/* We need to build a menu for these guys to live in. */
+		GtkMenu * menu = GTK_MENU(gtk_menu_new());
+		g_object_set_data(G_OBJECT(mi), data_menu, menu);
+
+		gtk_menu_item_set_submenu(gmi, GTK_WIDGET(menu));
+
+		g_signal_connect(menu, "notify::visible", G_CALLBACK(submenu_notify_visible_cb), mi);
+	}
+
+	return;
+}
+
 /* Whenever we have a property change on a DbusmenuMenuitem
    we need to be responsive to that. */
 static void
-menu_prop_change_cb (DbusmenuMenuitem * mi, gchar * prop, GVariant * variant, GtkMenuItem * gmi)
+menu_prop_change_cb (DbusmenuMenuitem * mi, gchar * prop, GVariant * variant, DbusmenuGtkClient * gtkclient)
 {
+	GtkMenuItem * gmi = dbusmenu_gtkclient_menuitem_get(gtkclient, mi);
+
 	if (!g_strcmp0(prop, DBUSMENU_MENUITEM_PROP_LABEL)) {
 		gtk_menu_item_set_label(gmi, variant == NULL ? NULL : g_variant_get_string(variant, NULL));
 	} else if (!g_strcmp0(prop, DBUSMENU_MENUITEM_PROP_VISIBLE)) {
@@ -566,6 +598,8 @@
 		process_toggle_type(mi, gmi, variant);
 	} else if (!g_strcmp0(prop, DBUSMENU_MENUITEM_PROP_TOGGLE_STATE)) {
 		process_toggle_state(mi, gmi, variant);
+	} else if (!g_strcmp0(prop, DBUSMENU_MENUITEM_PROP_CHILD_DISPLAY)) {
+		process_submenu(mi, gmi, variant, gtkclient);
 	}
 
 	return;
@@ -704,7 +738,7 @@
 	#endif
 
 	/* DbusmenuMenuitem signals */
-	g_signal_connect(G_OBJECT(item), DBUSMENU_MENUITEM_SIGNAL_PROPERTY_CHANGED, G_CALLBACK(menu_prop_change_cb), gmi);
+	g_signal_connect(G_OBJECT(item), DBUSMENU_MENUITEM_SIGNAL_PROPERTY_CHANGED, G_CALLBACK(menu_prop_change_cb), client);
 	g_signal_connect(G_OBJECT(item), DBUSMENU_MENUITEM_SIGNAL_PROPERTY_CHANGED, G_CALLBACK(menu_shortcut_change_cb), client);
 	g_signal_connect(G_OBJECT(item), DBUSMENU_MENUITEM_SIGNAL_CHILD_REMOVED, G_CALLBACK(delete_child), client);
 	g_signal_connect(G_OBJECT(item), DBUSMENU_MENUITEM_SIGNAL_CHILD_MOVED,   G_CALLBACK(move_child),   client);
@@ -720,6 +754,7 @@
 	process_sensitive(item, gmi, dbusmenu_menuitem_property_get_variant(item, DBUSMENU_MENUITEM_PROP_ENABLED));
 	process_toggle_type(item, gmi, dbusmenu_menuitem_property_get_variant(item, DBUSMENU_MENUITEM_PROP_TOGGLE_TYPE));
 	process_toggle_state(item, gmi, dbusmenu_menuitem_property_get_variant(item, DBUSMENU_MENUITEM_PROP_TOGGLE_STATE));
+	process_submenu(item, gmi, dbusmenu_menuitem_property_get_variant(item, DBUSMENU_MENUITEM_PROP_CHILD_DISPLAY), client);
 	refresh_shortcut(client, item);
 
 	/* Oh, we're a child, let's deal with that */
@@ -741,17 +776,12 @@
 	if (g_strcmp0(dbusmenu_menuitem_property_get(mi, DBUSMENU_MENUITEM_PROP_TYPE), DBUSMENU_CLIENT_TYPES_SEPARATOR) == 0) { return; }
 
 	gpointer ann_menu = g_object_get_data(G_OBJECT(mi), data_menu);
+	if (ann_menu == NULL) {
+		g_warning("Children but no menu, someone's been naughty with their '" DBUSMENU_MENUITEM_PROP_CHILD_DISPLAY "' property: '%s'", dbusmenu_menuitem_property_get(mi, DBUSMENU_MENUITEM_PROP_CHILD_DISPLAY));
+		return;
+	}
+
 	GtkMenu * menu = GTK_MENU(ann_menu);
-	if (menu == NULL) {
-		/* Oh, we don't have a submenu, build one! */
-		menu = GTK_MENU(gtk_menu_new());
-		g_object_set_data(G_OBJECT(mi), data_menu, menu);
-
-		GtkMenuItem * parent = dbusmenu_gtkclient_menuitem_get(gtkclient, mi);
-		gtk_menu_item_set_submenu(parent, GTK_WIDGET(menu));
-
-		g_signal_connect(menu, "notify::visible", G_CALLBACK(submenu_notify_visible_cb), mi);
-	} 
 
 	GtkMenuItem * childmi  = dbusmenu_gtkclient_menuitem_get(gtkclient, child);
 	gtk_menu_shell_insert(GTK_MENU_SHELL(menu), GTK_WIDGET(childmi), position);

=== modified file 'libdbusmenu-gtk/parser.c'
--- libdbusmenu-gtk/parser.c	2011-03-31 18:50:13 +0000
+++ libdbusmenu-gtk/parser.c	2011-04-07 15:38:25 +0000
@@ -79,6 +79,11 @@
                                                 gpointer            user_data);
 static gboolean       item_about_to_show       (DbusmenuMenuitem *  item,
                                                 gpointer            user_data);
+static gboolean       item_handle_event        (DbusmenuMenuitem *  item,
+                                                const gchar *       name,
+                                                GVariant *          variant,
+                                                guint               timestamp,
+                                                GtkWidget *         widget);
 static void           widget_notify_cb         (GtkWidget  *        widget,
                                                 GParamSpec *        pspec,
                                                 gpointer            data);
@@ -580,6 +585,11 @@
                             DBUSMENU_MENUITEM_SIGNAL_ABOUT_TO_SHOW,
                             G_CALLBACK (item_about_to_show),
                             widget);
+
+          g_signal_connect (G_OBJECT (mi),
+                            DBUSMENU_MENUITEM_SIGNAL_EVENT,
+                            G_CALLBACK (item_handle_event),
+                            widget);
         }
 
       dbusmenu_menuitem_property_set_bool (mi,
@@ -924,6 +934,34 @@
   return TRUE;
 }
 
+static gboolean
+item_handle_event (DbusmenuMenuitem *item, const gchar *name,
+                   GVariant *variant, guint timestamp, GtkWidget *widget)
+{
+  if (g_strcmp0 (name, DBUSMENU_MENUITEM_EVENT_OPENED) == 0)
+    {
+      GtkWidget *submenu = gtk_menu_item_get_submenu (GTK_MENU_ITEM (widget));
+      if (submenu != NULL)
+        {
+          // Show the submenu so the app can notice and futz with the menus as
+          // desired (empathy and geany do this)
+          gtk_widget_show (submenu);
+        }
+    }
+  else if (g_strcmp0 (name, DBUSMENU_MENUITEM_EVENT_CLOSED) == 0)
+    {
+      GtkWidget *submenu = gtk_menu_item_get_submenu (GTK_MENU_ITEM (widget));
+      if (submenu != NULL)
+        {
+          // Hide the submenu so the app can notice and futz with the menus as
+          // desired (empathy and geany do this)
+          gtk_widget_hide (submenu);
+        }
+    }
+
+  return FALSE; // just pass through on everything
+}
+
 static void
 widget_notify_cb (GtkWidget  *widget,
                   GParamSpec *pspec,