GtkMenuButton: rename _set_menu() to _set_popup()

It's too close to the release so we'll keep _set_menu() around, but
deprecate it, pointing to the new API.  It will be removed (and the name
reused) before the 3.6.0 release.

https://bugzilla.gnome.org/show_bug.cgi?id=682235

diff --git a/docs/reference/gtk/gtk3-sections.txt b/docs/reference/gtk/gtk3-sections.txt
index 8bdcb890db3433288acc57e1214e3b50afd01019..cf09ed94b0e21cbc59305751eb38184935b6f435 100644 (file)
--- a/docs/reference/gtk/gtk3-sections.txt
+++ b/docs/reference/gtk/gtk3-sections.txt
@@ -2130,8 +2130,8 @@ gtk_menu_bar_get_type
 <TITLE>GtkMenuButton</TITLE>
 GtkMenuButton
 gtk_menu_button_new
-gtk_menu_button_set_menu
-gtk_menu_button_get_menu
+gtk_menu_button_set_popup
+gtk_menu_button_get_popup
 gtk_menu_button_set_menu_model
 gtk_menu_button_get_menu_model
 gtk_menu_button_set_direction

diff --git a/docs/tools/widgets.c b/docs/tools/widgets.c
index 87e539eaeb6acf1007c486af7d0f7fe2d5266546..03c19eaa69757b2693df7c85b5e8371fa8d0cc77 100644 (file)
--- a/docs/tools/widgets.c
+++ b/docs/tools/widgets.c
@@ -237,7 +237,7 @@ create_menu_button (void)
   gtk_image_set_from_icon_name (GTK_IMAGE (image), "emblem-system-symbolic", GTK_ICON_SIZE_MENU);
   gtk_button_set_image (GTK_BUTTON (widget), image);
   menu = gtk_menu_new ();
-  gtk_menu_button_set_menu (GTK_MENU_BUTTON (widget), menu);
+  gtk_menu_button_set_popup (GTK_MENU_BUTTON (widget), menu);
 
   vbox = gtk_box_new (GTK_ORIENTATION_VERTICAL, 3);
   gtk_box_pack_start (GTK_BOX (vbox), widget, FALSE, FALSE, 0);

diff --git a/gtk/gtk.symbols b/gtk/gtk.symbols
index 1441671857ad7d7fc85107e20c258a7fdb59fdb4..84002789e8be40897bac2afb6ce4a37d75cea5f4 100644 (file)
--- a/gtk/gtk.symbols
+++ b/gtk/gtk.symbols
@@ -1551,16 +1551,16 @@ gtk_menu_bar_new
 gtk_menu_bar_new_from_model
 gtk_menu_bar_set_child_pack_direction
 gtk_menu_bar_set_pack_direction
-gtk_menu_button_get_type
-gtk_menu_button_new
 gtk_menu_button_get_align_widget
 gtk_menu_button_get_direction
-gtk_menu_button_get_menu
 gtk_menu_button_get_menu_model
+gtk_menu_button_get_popup
+gtk_menu_button_get_type
+gtk_menu_button_new
 gtk_menu_button_set_align_widget
 gtk_menu_button_set_direction
-gtk_menu_button_set_menu
 gtk_menu_button_set_menu_model
+gtk_menu_button_set_popup
 gtk_menu_detach
 gtk_menu_direction_type_get_type
 gtk_menu_get_accel_group

diff --git a/gtk/gtkmenubutton.c b/gtk/gtkmenubutton.c
index a1aaa8c3724c0e4de88aa6c1ae5c7d6e08080b4e..b324de599452451b28dc10c241e63a273918e331 100644 (file)
--- a/gtk/gtkmenubutton.c
+++ b/gtk/gtkmenubutton.c
@@ -151,7 +151,7 @@
 
 struct _GtkMenuButtonPrivate
 {
-  GtkWidget *menu;
+  GtkWidget *popup;
   GMenuModel *model;
 
   GtkMenuButtonShowMenuCallback func;
@@ -165,7 +165,7 @@ struct _GtkMenuButtonPrivate
 enum
 {
   PROP_0,
-  PROP_MENU,
+  PROP_POPUP,
   PROP_MODEL,
   PROP_ALIGN_WIDGET,
   PROP_DIRECTION
@@ -185,8 +185,8 @@ gtk_menu_button_set_property (GObject      *object,
 
   switch (property_id)
     {
-      case PROP_MENU:
-        gtk_menu_button_set_menu (self, g_value_get_object (value));
+      case PROP_POPUP:
+        gtk_menu_button_set_popup (self, g_value_get_object (value));
         break;
       case PROP_MODEL:
         gtk_menu_button_set_menu_model (self, g_value_get_object (value));
@@ -212,8 +212,8 @@ gtk_menu_button_get_property (GObject    *object,
 
   switch (property_id)
     {
-      case PROP_MENU:
-        g_value_set_object (value, priv->menu);
+      case PROP_POPUP:
+        g_value_set_object (value, priv->popup);
         break;
       case PROP_MODEL:
         g_value_set_object (value, priv->model);
@@ -236,8 +236,8 @@ gtk_menu_button_state_flags_changed (GtkWidget    *widget,
   GtkMenuButton *button = GTK_MENU_BUTTON (widget);
   GtkMenuButtonPrivate *priv = button->priv;
 
-  if (!gtk_widget_is_sensitive (widget) && priv->menu)
-    gtk_menu_shell_deactivate (GTK_MENU_SHELL (priv->menu));
+  if (!gtk_widget_is_sensitive (widget) && priv->popup)
+    gtk_menu_shell_deactivate (GTK_MENU_SHELL (priv->popup));
 }
 
 static void
@@ -258,10 +258,10 @@ menu_position_up_down_func (GtkMenu       *menu,
   GtkAllocation allocation, arrow_allocation;
   GtkAlign align;
 
-  gtk_widget_get_preferred_size (GTK_WIDGET (priv->menu),
+  gtk_widget_get_preferred_size (GTK_WIDGET (priv->popup),
                                  &menu_req, NULL);
 
-  align = gtk_widget_get_halign (GTK_WIDGET (priv->menu));
+  align = gtk_widget_get_halign (GTK_WIDGET (priv->popup));
   direction = gtk_widget_get_direction (widget);
   window = gtk_widget_get_window (priv->align_widget ? priv->align_widget : widget);
 
@@ -326,7 +326,7 @@ menu_position_side_func (GtkMenu       *menu,
   GdkWindow *window;
   GtkAlign align;
 
-  gtk_widget_get_preferred_size (GTK_WIDGET (priv->menu),
+  gtk_widget_get_preferred_size (GTK_WIDGET (priv->popup),
                                  &menu_req, NULL);
 
   window = gtk_widget_get_window (widget);
@@ -379,7 +379,7 @@ popup_menu (GtkMenuButton  *menu_button,
   if (priv->func)
     priv->func (priv->user_data);
 
-  if (!priv->menu)
+  if (!priv->popup)
     return;
 
   switch (priv->arrow_type)
@@ -393,7 +393,7 @@ popup_menu (GtkMenuButton  *menu_button,
         break;
   }
 
-  gtk_menu_popup_for_device (GTK_MENU (priv->menu),
+  gtk_menu_popup_for_device (GTK_MENU (priv->popup),
                              event ? event->device : NULL,
                              NULL, NULL,
                              func,
@@ -409,17 +409,17 @@ gtk_menu_button_toggled (GtkToggleButton *button)
   GtkMenuButton *menu_button = GTK_MENU_BUTTON (button);
   GtkMenuButtonPrivate *priv = menu_button->priv;
 
-  if (!priv->menu)
+  if (!priv->popup)
     return;
 
   if (gtk_toggle_button_get_active (button) &&
-      !gtk_widget_get_visible (GTK_WIDGET (priv->menu)))
+      !gtk_widget_get_visible (GTK_WIDGET (priv->popup)))
     {
       /* we get here only when the menu is activated by a key
        * press, so that we can select the first menu item
        */
       popup_menu (menu_button, NULL);
-      gtk_menu_shell_select_first (GTK_MENU_SHELL (priv->menu), FALSE);
+      gtk_menu_shell_select_first (GTK_MENU_SHELL (priv->popup), FALSE);
     }
 }
 
@@ -457,19 +457,35 @@ gtk_menu_button_class_init (GtkMenuButtonClass *klass)
   toggle_button_class->toggled = gtk_menu_button_toggled;
 
   /**
+   * GtkMenuButton:popup:
+   *
+   * The #GtkMenu that will be popped up when the button is clicked.
+   *
+   * Since: 3.6
+   */
+  g_object_class_install_property (gobject_class,
+                                   PROP_POPUP,
+                                   g_param_spec_object ("popup",
+                                                        P_("popup"),
+                                                        P_("The dropdown menu."),
+                                                        GTK_TYPE_MENU,
+                                                        G_PARAM_READWRITE));
+   /**
    * GtkMenuButton:menu:
    *
    * The #GtkMenu that will be popped up when the button is clicked.
+   * This property has been renamed to "popup".  "menu" will be
+   * removed before 3.6.0.
    *
    * Since: 3.6
    */
   g_object_class_install_property (gobject_class,
-                                   PROP_MENU,
+                                   PROP_POPUP, /* [sic] */
                                    g_param_spec_object ("menu",
                                                         P_("menu"),
                                                         P_("The dropdown menu."),
                                                         GTK_TYPE_MENU,
-                                                        G_PARAM_READWRITE));
+                                                        G_PARAM_DEPRECATED | G_PARAM_READWRITE));
   /**
    * GtkMenuButton:menu-model:
    *
@@ -579,9 +595,9 @@ menu_detacher (GtkWidget *widget,
 {
   GtkMenuButtonPrivate *priv = GTK_MENU_BUTTON (widget)->priv;
 
-  g_return_if_fail (priv->menu == (GtkWidget *) menu);
+  g_return_if_fail (priv->popup == (GtkWidget *) menu);
 
-  priv->menu = NULL;
+  priv->popup = NULL;
 }
 
 /* This function is used in GtkMenuToolButton, the call back will
@@ -589,10 +605,10 @@ menu_detacher (GtkWidget *widget,
  * signal.
  */
 void
-_gtk_menu_button_set_menu_with_func (GtkMenuButton                 *menu_button,
-                                     GtkWidget                     *menu,
-                                     GtkMenuButtonShowMenuCallback  func,
-                                     gpointer                       user_data)
+_gtk_menu_button_set_popup_with_func (GtkMenuButton                 *menu_button,
+                                      GtkWidget                     *menu,
+                                      GtkMenuButtonShowMenuCallback  func,
+                                      gpointer                       user_data)
 {
   GtkMenuButtonPrivate *priv;
 
@@ -603,33 +619,33 @@ _gtk_menu_button_set_menu_with_func (GtkMenuButton                 *menu_button,
   priv->func = func;
   priv->user_data = user_data;
 
-  if (priv->menu == GTK_WIDGET (menu))
+  if (priv->popup == GTK_WIDGET (menu))
     return;
 
-  if (priv->menu)
+  if (priv->popup)
     {
-      if (gtk_widget_get_visible (GTK_WIDGET (priv->menu)))
-        gtk_menu_shell_deactivate (GTK_MENU_SHELL (priv->menu));
+      if (gtk_widget_get_visible (GTK_WIDGET (priv->popup)))
+        gtk_menu_shell_deactivate (GTK_MENU_SHELL (priv->popup));
     }
 
-  if (priv->menu)
+  if (priv->popup)
     {
-      g_signal_handlers_disconnect_by_func (priv->menu,
+      g_signal_handlers_disconnect_by_func (priv->popup,
                                             menu_deactivate_cb,
                                             menu_button);
-      gtk_menu_detach (GTK_MENU (priv->menu));
+      gtk_menu_detach (GTK_MENU (priv->popup));
     }
 
-  priv->menu = menu;
+  priv->popup = menu;
 
-  if (priv->menu)
+  if (priv->popup)
     {
-      gtk_menu_attach_to_widget (GTK_MENU (priv->menu), GTK_WIDGET (menu_button),
+      gtk_menu_attach_to_widget (GTK_MENU (priv->popup), GTK_WIDGET (menu_button),
                                  menu_detacher);
 
       gtk_widget_set_sensitive (GTK_WIDGET (menu_button), TRUE);
 
-      g_signal_connect (priv->menu, "deactivate",
+      g_signal_connect (priv->popup, "deactivate",
                         G_CALLBACK (menu_deactivate_cb), menu_button);
     }
   else
@@ -642,9 +658,9 @@ _gtk_menu_button_set_menu_with_func (GtkMenuButton                 *menu_button,
 }
 
 /**
- * gtk_menu_button_set_menu:
+ * gtk_menu_button_set_popup:
  * @menu_button: a #GtkMenuButton
- * @menu: (allow-none): a #GtkMenu
+ * @popup: (allow-none): a #GtkMenu
  *
  * Sets the #GtkMenu that will be popped up when the button is clicked,
  * or %NULL to disable the button. If #GtkMenuButton:menu-model is set,
@@ -653,22 +669,22 @@ _gtk_menu_button_set_menu_with_func (GtkMenuButton                 *menu_button,
  * Since: 3.6
  */
 void
-gtk_menu_button_set_menu (GtkMenuButton *menu_button,
-                          GtkWidget     *menu)
+gtk_menu_button_set_popup (GtkMenuButton *menu_button,
+                           GtkWidget     *popup)
 {
   GtkMenuButtonPrivate *priv;
 
   g_return_if_fail (GTK_IS_MENU_BUTTON (menu_button));
-  g_return_if_fail (GTK_IS_MENU (menu) || menu == NULL);
+  g_return_if_fail (GTK_IS_MENU (popup) || popup == NULL);
 
   priv = menu_button->priv;
   g_clear_object (&priv->model);
 
-  _gtk_menu_button_set_menu_with_func (menu_button, menu, NULL, NULL);
+  _gtk_menu_button_set_popup_with_func (menu_button, popup, NULL, NULL);
 }
 
 /**
- * gtk_menu_button_get_menu:
+ * gtk_menu_button_get_popup:
  * @menu_button: a #GtkMenuButton
  *
  * Returns the #GtkMenu that pops out of the button.
@@ -678,11 +694,24 @@ gtk_menu_button_set_menu (GtkMenuButton *menu_button,
  * Since: 3.6
  */
 GtkMenu *
-gtk_menu_button_get_menu (GtkMenuButton *menu_button)
+gtk_menu_button_get_popup (GtkMenuButton *menu_button)
 {
   g_return_val_if_fail (GTK_IS_MENU_BUTTON (menu_button), NULL);
 
-  return GTK_MENU (menu_button->priv->menu);
+  return GTK_MENU (menu_button->priv->popup);
+}
+
+void
+gtk_menu_button_set_menu (GtkMenuButton *menu_button,
+                          GtkWidget     *menu)
+{
+  gtk_menu_button_set_popup (menu_button, menu);
+}
+
+GtkMenu *
+gtk_menu_button_get_menu (GtkMenuButton *menu_button)
+{
+  return gtk_menu_button_get_popup (menu_button);
 }
 
 /**
@@ -716,14 +745,14 @@ gtk_menu_button_set_menu_model (GtkMenuButton *menu_button,
 
   if (menu_model == NULL)
     {
-      gtk_menu_button_set_menu (menu_button, NULL);
+      gtk_menu_button_set_popup (menu_button, NULL);
       return;
     }
 
   priv->model = g_object_ref (menu_model);
   menu = gtk_menu_new_from_model (menu_model);
   gtk_widget_show_all (menu);
-  gtk_menu_button_set_menu (menu_button, menu);
+  gtk_menu_button_set_popup (menu_button, menu);
 }
 
 /**
@@ -858,12 +887,12 @@ gtk_menu_button_dispose (GObject *object)
 {
   GtkMenuButtonPrivate *priv = GTK_MENU_BUTTON (object)->priv;
 
-  if (priv->menu)
+  if (priv->popup)
     {
-      g_signal_handlers_disconnect_by_func (priv->menu,
+      g_signal_handlers_disconnect_by_func (priv->popup,
                                             menu_deactivate_cb,
                                             object);
-      gtk_menu_detach (GTK_MENU (priv->menu));
+      gtk_menu_detach (GTK_MENU (priv->popup));
     }
 
   g_clear_object (&priv->model);

diff --git a/gtk/gtkmenubutton.h b/gtk/gtkmenubutton.h
index 3ac6faa3da5e366c833742eb9638610201f6a1e6..f03e8a9bbdf9facae9ee7aa8c0b3a593860e60f9 100644 (file)
--- a/gtk/gtkmenubutton.h
+++ b/gtk/gtkmenubutton.h
@@ -65,9 +65,15 @@ GDK_AVAILABLE_IN_3_6
 GtkWidget   *gtk_menu_button_new            (void);
 
 GDK_AVAILABLE_IN_3_6
-void         gtk_menu_button_set_menu       (GtkMenuButton *menu_button,
+void         gtk_menu_button_set_popup      (GtkMenuButton *menu_button,
                                              GtkWidget     *menu);
 GDK_AVAILABLE_IN_3_6
+GtkMenu     *gtk_menu_button_get_popup      (GtkMenuButton *menu_button);
+
+GDK_DEPRECATED_IN_3_6_FOR(gtk_menu_button_set_popup)
+void         gtk_menu_button_set_menu       (GtkMenuButton *menu_button,
+                                             GtkWidget     *menu);
+GDK_DEPRECATED_IN_3_6_FOR(gtk_menu_button_get_popup)
 GtkMenu     *gtk_menu_button_get_menu       (GtkMenuButton *menu_button);
 
 GDK_AVAILABLE_IN_3_6

diff --git a/gtk/gtkmenubuttonprivate.h b/gtk/gtkmenubuttonprivate.h
index 0de73470a2c440aec0c4e4c78b60b082bf560a52..982ad2bdf92c35cd364268f7e8fab0c333d12cf3 100644 (file)
--- a/gtk/gtkmenubuttonprivate.h
+++ b/gtk/gtkmenubuttonprivate.h
@@ -27,10 +27,10 @@ G_BEGIN_DECLS
 
 typedef void (* GtkMenuButtonShowMenuCallback) (gpointer user_data);
 
-void       _gtk_menu_button_set_menu_with_func (GtkMenuButton                 *menu_button,
-                                                GtkWidget                     *menu,
-                                                GtkMenuButtonShowMenuCallback  func,
-                                                gpointer                       user_data);
+void       _gtk_menu_button_set_popup_with_func (GtkMenuButton                 *menu_button,
+                                                 GtkWidget                     *menu,
+                                                 GtkMenuButtonShowMenuCallback  func,
+                                                 gpointer                       user_data);
 
 G_END_DECLS
 

diff --git a/gtk/gtkmenutoolbutton.c b/gtk/gtkmenutoolbutton.c
index c7d014ca4d5c6d74b5228792bc7fcd020893ec98..c133f88a7b5bc10e63dfa8fe5a754719516b3b5a 100644 (file)
--- a/gtk/gtkmenutoolbutton.c
+++ b/gtk/gtkmenutoolbutton.c
@@ -221,7 +221,7 @@ gtk_menu_tool_button_get_property (GObject    *object,
   switch (prop_id)
     {
     case PROP_MENU:
-      g_value_set_object (value, gtk_menu_button_get_menu (GTK_MENU_BUTTON (button->priv->arrow_button)));
+      g_value_set_object (value, gtk_menu_button_get_popup (GTK_MENU_BUTTON (button->priv->arrow_button)));
       break;
 
     default:
@@ -419,10 +419,10 @@ gtk_menu_tool_button_set_menu (GtkMenuToolButton *button,
 
   priv = button->priv;
 
-  _gtk_menu_button_set_menu_with_func (GTK_MENU_BUTTON (priv->arrow_button),
-                                       menu,
-                                       _show_menu_emit,
-                                       button);
+  _gtk_menu_button_set_popup_with_func (GTK_MENU_BUTTON (priv->arrow_button),
+                                        menu,
+                                        _show_menu_emit,
+                                        button);
 
   g_object_notify (G_OBJECT (button), "menu");
 }
@@ -445,7 +445,7 @@ gtk_menu_tool_button_get_menu (GtkMenuToolButton *button)
 
   g_return_val_if_fail (GTK_IS_MENU_TOOL_BUTTON (button), NULL);
 
-  ret = gtk_menu_button_get_menu (GTK_MENU_BUTTON (button->priv->arrow_button));
+  ret = gtk_menu_button_get_popup (GTK_MENU_BUTTON (button->priv->arrow_button));
   if (!ret)
     return NULL;
 

diff --git a/tests/testmenubutton.c b/tests/testmenubutton.c
index cf48b2e5ec8937e82970b9f9bef0975bed529c3a..b7d5b5265b64ee2f76eb57d482f2ecca3506096c 100644 (file)
--- a/tests/testmenubutton.c
+++ b/tests/testmenubutton.c
@@ -48,7 +48,7 @@ int main (int argc, char **argv)
        gtk_widget_show_all (menu_widget);
 
        button = gtk_menu_button_new ();
-       gtk_menu_button_set_menu (GTK_MENU_BUTTON (button), menu_widget);
+       gtk_menu_button_set_popup (GTK_MENU_BUTTON (button), menu_widget);
        gtk_grid_attach (GTK_GRID (grid),
                         button,
                         1, 1,