Looks good

  review approve

On Thu, 2010-03-11 at 20:26 +0000, Cody Russell wrote:
> Cody Russell has proposed merging lp:~bratsche/indicator-me/auto-select-entry into lp:indicator-me.
> 
> Requested reviews:
>   Indicator Applet Developers (indicator-applet-developers)
> Related bugs:
>   #528302 The text field should receive focus automatically
>   https://bugs.launchpad.net/bugs/528302
> 
> differences between files attachment (review-diff.txt)
> === modified file 'src/indicator-me.c'
> --- src/indicator-me.c	2010-03-11 17:10:36 +0000
> +++ src/indicator-me.c	2010-03-11 20:26:16 +0000
> @@ -7,17 +7,18 @@
>  
>    Authors:
>    Ted Gould <email address hidden>
> +  Cody Russell <email address hidden>
>  
> -  This program is free software: you can redistribute it and/or modify it 
> -  under the terms of the GNU General Public License version 3, as published 
> +  This program is free software: you can redistribute it and/or modify it
> +  under the terms of the GNU General Public License version 3, as published
>    by the Free Software Foundation.
>  
> -  This program is distributed in the hope that it will be useful, but 
> -  WITHOUT ANY WARRANTY; without even the implied warranties of 
> -  MERCHANTABILITY, SATISFACTORY QUALITY, or FITNESS FOR A PARTICULAR 
> +  This program is distributed in the hope that it will be useful, but
> +  WITHOUT ANY WARRANTY; without even the implied warranties of
> +  MERCHANTABILITY, SATISFACTORY QUALITY, or FITNESS FOR A PARTICULAR
>    PURPOSE.  See the GNU General Public License for more details.
>  
> -  You should have received a copy of the GNU General Public License along 
> +  You should have received a copy of the GNU General Public License along
>    with this program.  If not, see <http://www.gnu.org/licenses/>.
>  */
>  
> @@ -253,9 +254,32 @@
>  }
>  
>  static gboolean
> +menu_visibility_changed (GtkWidget          *widget,
> +                         IdoEntryMenuItem   *menuitem)
> +{
> +  gtk_menu_shell_select_item (GTK_MENU_SHELL (widget), GTK_WIDGET (menuitem));
> +
> +  return FALSE;
> +}
> +
> +static void
> +entry_parent_changed (GtkWidget *widget,
> +                      gpointer   user_data)
> +{
> +  GtkWidget *parent = gtk_widget_get_parent (widget);
> +
> +  if (parent && GTK_IS_MENU_SHELL (parent))
> +    {
> +      g_signal_connect (parent,
> +                        "map", G_CALLBACK (menu_visibility_changed),
> +                        widget);
> +    }
> +}
> +
> +static gboolean
>  new_entry_item (DbusmenuMenuitem * newitem,
>                  DbusmenuMenuitem * parent,
> -                DbusmenuClient * client)
> +                DbusmenuClient   * client)
>  {
>  	g_return_val_if_fail(DBUSMENU_IS_MENUITEM(newitem), FALSE);
>  	g_return_val_if_fail(DBUSMENU_IS_GTKCLIENT(client), FALSE);
> @@ -267,6 +291,10 @@
>  		gtk_entry_set_text(entry, dbusmenu_menuitem_property_get(newitem, DBUSMENU_ENTRY_MENUITEM_PROP_TEXT));
>  	gtk_entry_set_width_chars (entry, 23); /* set some nice aspect ratio for the menu */
>  
> +  g_signal_connect (ido,
> +                    "notify::parent", G_CALLBACK (entry_parent_changed),
> +                    NULL);
> +
>  	dbusmenu_gtkclient_newitem_base(DBUSMENU_GTKCLIENT(client), newitem, GTK_MENU_ITEM(ido), parent);
>  	/* disconnect the activate signal that newitem_base connected with the wrong
>  	   widget, ie menuitem, and re-connect it with the /entry/ instead */
>