[gtk+/wip/attachment: 3/3] gtkmenu: add gtk_menu_popup_with_parameters ()
- From: William Hua <williamhua src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gtk+/wip/attachment: 3/3] gtkmenu: add gtk_menu_popup_with_parameters ()
- Date: Sun, 13 Sep 2015 23:22:19 +0000 (UTC)
commit 9ce97509245e2bc8185076bd2bf56f53f42edacf
Author: William Hua <william hua canonical com>
Date: Sat Sep 12 10:51:54 2015 -0400
gtkmenu: add gtk_menu_popup_with_parameters ()
gtk/gtkmenu.c | 201 +++++++++++++++++++++++++++++++++++--------------
gtk/gtkmenu.h | 40 ++++++++++
gtk/gtkmenuprivate.h | 2 +
3 files changed, 185 insertions(+), 58 deletions(-)
---
diff --git a/gtk/gtkmenu.c b/gtk/gtkmenu.c
index b39de7d..e97656f 100644
--- a/gtk/gtkmenu.c
+++ b/gtk/gtkmenu.c
@@ -1152,6 +1152,8 @@ gtk_menu_destroy (GtkWidget *widget)
g_clear_pointer (&priv->title, g_free);
+ g_clear_pointer (&priv->attachment_parameters, gdk_attachment_parameters_free);
+
if (priv->position_func_data_destroy)
{
priv->position_func_data_destroy (priv->position_func_data);
@@ -1496,53 +1498,17 @@ associate_menu_grab_transfer_window (GtkMenu *menu)
g_object_set_data (G_OBJECT (toplevel_window), I_("gdk-attached-grab-window"), transfer_window);
}
-/**
- * gtk_menu_popup_for_device:
- * @menu: a #GtkMenu
- * @device: (allow-none): a #GdkDevice
- * @parent_menu_shell: (allow-none): the menu shell containing the triggering
- * menu item, or %NULL
- * @parent_menu_item: (allow-none): the menu item whose activation triggered
- * the popup, or %NULL
- * @func: (allow-none): a user supplied function used to position the menu,
- * or %NULL
- * @data: (allow-none): user supplied data to be passed to @func
- * @destroy: (allow-none): destroy notify for @data
- * @button: the mouse button which was pressed to initiate the event
- * @activate_time: the time at which the activation event occurred
- *
- * Displays a menu and makes it available for selection.
- *
- * Applications can use this function to display context-sensitive menus,
- * and will typically supply %NULL for the @parent_menu_shell,
- * @parent_menu_item, @func, @data and @destroy parameters. The default
- * menu positioning function will position the menu at the current position
- * of @device (or its corresponding pointer).
- *
- * The @button parameter should be the mouse button pressed to initiate
- * the menu popup. If the menu popup was initiated by something other than
- * a mouse button press, such as a mouse button release or a keypress,
- * @button should be 0.
- *
- * The @activate_time parameter is used to conflict-resolve initiation of
- * concurrent requests for mouse/keyboard grab requests. To function
- * properly, this needs to be the time stamp of the user event (such as
- * a mouse click or key press) that caused the initiation of the popup.
- * Only if no such event is available, gtk_get_current_event_time() can
- * be used instead.
- *
- * Since: 3.0
- */
-void
-gtk_menu_popup_for_device (GtkMenu *menu,
- GdkDevice *device,
- GtkWidget *parent_menu_shell,
- GtkWidget *parent_menu_item,
- GtkMenuPositionFunc func,
- gpointer data,
- GDestroyNotify destroy,
- guint button,
- guint32 activate_time)
+static void
+gtk_menu_popup_internal (GtkMenu *menu,
+ GdkDevice *device,
+ GtkWidget *parent_menu_shell,
+ GtkWidget *parent_menu_item,
+ GtkMenuPositionFunc func,
+ gpointer data,
+ GDestroyNotify destroy,
+ guint button,
+ guint32 activate_time,
+ GdkAttachmentParameters *parameters)
{
GtkMenuPrivate *priv = menu->priv;
GtkWidget *widget;
@@ -1555,6 +1521,9 @@ gtk_menu_popup_for_device (GtkMenu *menu,
GdkDevice *keyboard, *pointer, *source_device = NULL;
GdkDisplay *display;
+ if (!GTK_IS_MENU (menu) || (device && !GDK_IS_DEVICE (device)))
+ gdk_attachment_parameters_free (parameters);
+
g_return_if_fail (GTK_IS_MENU (menu));
g_return_if_fail (device == NULL || GDK_IS_DEVICE (device));
@@ -1674,6 +1643,7 @@ gtk_menu_popup_for_device (GtkMenu *menu,
*/
menu_shell->priv->parent_menu_shell = NULL;
menu_grab_transfer_window_destroy (menu);
+ gdk_attachment_parameters_free (parameters);
return;
}
@@ -1727,6 +1697,14 @@ gtk_menu_popup_for_device (GtkMenu *menu,
priv->position_func_data_destroy = destroy;
menu_shell->priv->activate_time = activate_time;
+ if (!parameters || parameters != priv->attachment_parameters)
+ {
+ gdk_attachment_parameters_free (priv->attachment_parameters);
+ priv->attachment_parameters = parameters;
+ }
+ else
+ g_warning ("gtk_menu_popup_internal (): menu already owns attachment parameters");
+
/* We need to show the menu here rather in the init function
* because code expects to be able to tell if the menu is onscreen
* by looking at gtk_widget_get_visible (menu)
@@ -1767,6 +1745,108 @@ gtk_menu_popup_for_device (GtkMenu *menu,
_gtk_menu_shell_update_mnemonics (menu_shell);
}
+GdkAttachmentParameters *
+gtk_menu_attachment_parameters (GtkMenu *menu,
+ GtkWidget *parent_menu_shell,
+ GtkWidget *parent_menu_item,
+ const GdkRectangle *attachment_rectangle,
+ GtkAttachmentOptions attachment_options,
+ const GdkPoint *attachment_offset,
+ GdkAttachmentOffsetCallback attachment_callback)
+{
+ GdkAttachmentParameters *parameters = gdk_attachment_parameters_new ();
+
+ parameters->offset_callback = attachment_callback;
+
+ return parameters;
+}
+
+void
+gtk_menu_popup_with_parameters (GtkMenu *menu,
+ GdkDevice *device,
+ guint button,
+ guint32 activate_time,
+ GdkAttachmentParameters *parameters)
+{
+ if (!GTK_IS_MENU (menu))
+ gdk_attachment_parameters_free (parameters);
+
+ g_return_if_fail (GTK_IS_MENU (menu));
+
+ gtk_menu_popup_internal (menu,
+ device,
+ NULL,
+ NULL,
+ NULL,
+ NULL,
+ NULL,
+ button,
+ activate_time,
+ parameters);
+}
+
+/**
+ * gtk_menu_popup_for_device:
+ * @menu: a #GtkMenu
+ * @device: (allow-none): a #GdkDevice
+ * @parent_menu_shell: (allow-none): the menu shell containing the triggering
+ * menu item, or %NULL
+ * @parent_menu_item: (allow-none): the menu item whose activation triggered
+ * the popup, or %NULL
+ * @func: (allow-none): a user supplied function used to position the menu,
+ * or %NULL
+ * @data: (allow-none): user supplied data to be passed to @func
+ * @destroy: (allow-none): destroy notify for @data
+ * @button: the mouse button which was pressed to initiate the event
+ * @activate_time: the time at which the activation event occurred
+ *
+ * Displays a menu and makes it available for selection.
+ *
+ * Applications can use this function to display context-sensitive menus,
+ * and will typically supply %NULL for the @parent_menu_shell,
+ * @parent_menu_item, @func, @data and @destroy parameters. The default
+ * menu positioning function will position the menu at the current position
+ * of @device (or its corresponding pointer).
+ *
+ * The @button parameter should be the mouse button pressed to initiate
+ * the menu popup. If the menu popup was initiated by something other than
+ * a mouse button press, such as a mouse button release or a keypress,
+ * @button should be 0.
+ *
+ * The @activate_time parameter is used to conflict-resolve initiation of
+ * concurrent requests for mouse/keyboard grab requests. To function
+ * properly, this needs to be the time stamp of the user event (such as
+ * a mouse click or key press) that caused the initiation of the popup.
+ * Only if no such event is available, gtk_get_current_event_time() can
+ * be used instead.
+ *
+ * Since: 3.0
+ */
+void
+gtk_menu_popup_for_device (GtkMenu *menu,
+ GdkDevice *device,
+ GtkWidget *parent_menu_shell,
+ GtkWidget *parent_menu_item,
+ GtkMenuPositionFunc func,
+ gpointer data,
+ GDestroyNotify destroy,
+ guint button,
+ guint32 activate_time)
+{
+ g_return_if_fail (GTK_IS_MENU (menu));
+
+ gtk_menu_popup_internal (menu,
+ device,
+ parent_menu_shell,
+ parent_menu_item,
+ func,
+ data,
+ destroy,
+ button,
+ activate_time,
+ NULL);
+}
+
/**
* gtk_menu_popup:
* @menu: a #GtkMenu
@@ -4471,17 +4551,28 @@ gtk_menu_position (GtkMenu *menu,
GdkDevice *pointer;
GtkBorder border;
+ /* Realize so we have the proper width and height to figure out
+ * the right place to popup the menu.
+ */
+ gtk_widget_realize (priv->toplevel);
+
+ /* Set the type hint here to allow custom position functions
+ * to set a different hint
+ */
+ if (!gtk_widget_get_visible (priv->toplevel))
+ gtk_window_set_type_hint (GTK_WINDOW (priv->toplevel), GDK_WINDOW_TYPE_HINT_POPUP_MENU);
+
+ gdk_window_set_attachment_parameters (gtk_widget_get_window (priv->toplevel), priv->attachment_parameters);
+
+ if (priv->attachment_parameters)
+ return;
+
widget = GTK_WIDGET (menu);
screen = gtk_widget_get_screen (widget);
pointer = _gtk_menu_shell_get_grab_device (GTK_MENU_SHELL (menu));
gdk_device_get_position (pointer, &pointer_screen, &x, &y);
- /* Realize so we have the proper width and height to figure out
- * the right place to popup the menu.
- */
- gtk_widget_realize (priv->toplevel);
-
_gtk_window_get_shadow_width (GTK_WINDOW (priv->toplevel), &border);
requisition.width = gtk_widget_get_allocated_width (widget);
@@ -4501,12 +4592,6 @@ gtk_menu_position (GtkMenu *menu,
priv->monitor_num = gdk_screen_get_monitor_at_point (screen, x, y);
priv->initially_pushed_in = FALSE;
- /* Set the type hint here to allow custom position functions
- * to set a different hint
- */
- if (!gtk_widget_get_visible (priv->toplevel))
- gtk_window_set_type_hint (GTK_WINDOW (priv->toplevel), GDK_WINDOW_TYPE_HINT_POPUP_MENU);
-
if (priv->position_func)
{
(* priv->position_func) (menu, &x, &y, &priv->initially_pushed_in,
diff --git a/gtk/gtkmenu.h b/gtk/gtkmenu.h
index bf81a89..b8db725 100644
--- a/gtk/gtkmenu.h
+++ b/gtk/gtkmenu.h
@@ -155,6 +155,46 @@ void gtk_menu_popup_for_device (GtkMenu *menu,
guint button,
guint32 activate_time);
+typedef enum _GtkAttachmentOptions GtkAttachmentOptions;
+
+enum _GtkAttachmentOptions
+{
+ GTK_ATTACHMENT_ATTACH_NO_EDGE = 0x1,
+ GTK_ATTACHMENT_ATTACH_TOP_EDGE = 0x2,
+ GTK_ATTACHMENT_ATTACH_LEFT_EDGE = 0x3,
+ GTK_ATTACHMENT_ATTACH_RIGHT_EDGE = 0x4,
+ GTK_ATTACHMENT_ATTACH_BOTTOM_EDGE = 0x5,
+ GTK_ATTACHMENT_ATTACH_ANY_EDGE = 0x6,
+ GTK_ATTACHMENT_ATTACH_MASK = 0x7,
+ GTK_ATTACHMENT_ATTACH_OPPOSITE_EDGE = 0x8,
+ GTK_ATTACHMENT_ALIGN_TOP_EDGES = 0x10,
+ GTK_ATTACHMENT_ALIGN_LEFT_EDGES = 0x20,
+ GTK_ATTACHMENT_ALIGN_RIGHT_EDGES = 0x40,
+ GTK_ATTACHMENT_ALIGN_BOTTOM_EDGES = 0x80,
+ GTK_ATTACHMENT_ALIGN_VERTICAL_MASK = 0x90,
+ GTK_ATTACHMENT_ALIGN_HORIZONTAL_MASK = 0x60,
+ GTK_ATTACHMENT_ALIGN_MASK = 0xF0,
+ GTK_ATTACHMENT_ALIGN_VERTICALLY = 0x100,
+ GTK_ATTACHMENT_ALIGN_HORIZONTALLY = 0x200
+};
+
+GDK_AVAILABLE_IN_3_18
+GdkAttachmentParameters *
+ gtk_menu_attachment_parameters (GtkMenu *menu,
+ GtkWidget *parent_menu_shell,
+ GtkWidget *parent_menu_item,
+ const GdkRectangle *attachment_rectangle,
+ GtkAttachmentOptions attachment_options,
+ const GdkPoint *attachment_offset,
+ GdkAttachmentOffsetCallback attachment_callback);
+
+GDK_AVAILABLE_IN_3_18
+void gtk_menu_popup_with_parameters (GtkMenu *menu,
+ GdkDevice *device,
+ guint button,
+ guint32 activate_time,
+ GdkAttachmentParameters *parameters);
+
/* Position the menu according to its position function. Called
* from gtkmenuitem.c when a menu-item changes its allocation
*/
diff --git a/gtk/gtkmenuprivate.h b/gtk/gtkmenuprivate.h
index f1a40d8..5e0295d 100644
--- a/gtk/gtkmenuprivate.h
+++ b/gtk/gtkmenuprivate.h
@@ -51,6 +51,8 @@ struct _GtkMenuPrivate
gint position_x;
gint position_y;
+ GdkAttachmentParameters *attachment_parameters;
+
guint toggle_size;
guint accel_size;
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]
