[gtk+/composite-templates] GtkMenuButton: rename _set_menu() to _set_popup()
- From: Juan Pablo Ugarte <jpu src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gtk+/composite-templates] GtkMenuButton: rename _set_menu() to _set_popup()
- Date: Fri, 14 Sep 2012 21:15:29 +0000 (UTC)
commit 9f4d35f26cbc0f0f4397379dbda1237fbb78c2c3
Author: Ryan Lortie <desrt desrt ca>
Date: Mon Aug 20 17:03:15 2012 -0400
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
docs/reference/gtk/gtk3-sections.txt | 4 +-
docs/tools/widgets.c | 2 +-
gtk/gtk.symbols | 8 +-
gtk/gtkmenubutton.c | 127 +++++++++++++++++++++-------------
gtk/gtkmenubutton.h | 8 ++-
gtk/gtkmenubuttonprivate.h | 8 +-
gtk/gtkmenutoolbutton.c | 12 ++--
tests/testmenubutton.c | 2 +-
8 files changed, 103 insertions(+), 68 deletions(-)
---
diff --git a/docs/reference/gtk/gtk3-sections.txt b/docs/reference/gtk/gtk3-sections.txt
index 8bdcb89..cf09ed9 100644
--- 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 87e539e..03c19ea 100644
--- 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 1441671..8400278 100644
--- 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 a1aaa8c..b324de5 100644
--- 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 3ac6faa..f03e8a9 100644
--- 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 0de7347..982ad2b 100644
--- 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 c7d014c..c133f88 100644
--- 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 cf48b2e..b7d5b52 100644
--- 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,
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]
