[gnome-photos] preview-nav-buttons: Improve long press detection

[Debarshi Ray <debarshir src gnome org>]

commit f19cd64e2aead22f1482ea838a845c9ab6de589b
Author: Debarshi Ray <debarshir gnome org>
Date:   Sun Nov 27 13:14:08 2016 +0100

    preview-nav-buttons: Improve long press detection
    
    GtkGestureMultiPress::stopped is emitted, and it is meant to be,
    regardless of whether it was a tap or a long press. It is better to use
    GtkGestureLongPress::pressed to clearly identify long presses.

 src/photos-preview-nav-buttons.c |   28 +++++++++++++++++-----------
 1 files changed, 17 insertions(+), 11 deletions(-)
---
diff --git a/src/photos-preview-nav-buttons.c b/src/photos-preview-nav-buttons.c
index d27295c..28c9751 100644
--- a/src/photos-preview-nav-buttons.c
+++ b/src/photos-preview-nav-buttons.c
@@ -42,6 +42,7 @@ struct _PhotosPreviewNavButtons
   GObject parent_instance;
   GAction *load_next;
   GAction *load_previous;
+  GtkGesture *long_press_gesture;
   GtkGesture *tap_gesture;
   GtkTreeModel *model;
   GtkTreeRowReference *current_row;
@@ -273,19 +274,19 @@ photos_preview_nav_buttons_motion_notify (PhotosPreviewNavButtons *self, GdkEven
 
 
 static void
-photos_preview_nav_buttons_multi_press_released (PhotosPreviewNavButtons *self)
+photos_preview_nav_buttons_long_press_pressed (PhotosPreviewNavButtons *self)
 {
-  gtk_gesture_set_state (GTK_GESTURE (self->tap_gesture), GTK_EVENT_SEQUENCE_CLAIMED);
-  self->visible_internal = !self->visible_internal;
-  photos_preview_nav_buttons_unqueue_auto_hide (self);
-  photos_preview_nav_buttons_update_visibility (self);
+  gtk_gesture_set_state (GTK_GESTURE (self->long_press_gesture), GTK_EVENT_SEQUENCE_DENIED);
 }
 
 
 static void
-photos_preview_nav_buttons_multi_press_stopped (PhotosPreviewNavButtons *self)
+photos_preview_nav_buttons_multi_press_released (PhotosPreviewNavButtons *self)
 {
-  gtk_gesture_set_state (GTK_GESTURE (self->tap_gesture), GTK_EVENT_SEQUENCE_DENIED);
+  gtk_gesture_set_state (GTK_GESTURE (self->tap_gesture), GTK_EVENT_SEQUENCE_CLAIMED);
+  self->visible_internal = !self->visible_internal;
+  photos_preview_nav_buttons_unqueue_auto_hide (self);
+  photos_preview_nav_buttons_update_visibility (self);
 }
 
 
@@ -365,6 +366,7 @@ photos_preview_nav_buttons_dispose (GObject *object)
 
   photos_preview_nav_buttons_unqueue_auto_hide (self);
 
+  g_clear_object (&self->long_press_gesture);
   g_clear_object (&self->tap_gesture);
   g_clear_object (&self->model);
   g_clear_object (&self->next_widget);
@@ -454,16 +456,20 @@ photos_preview_nav_buttons_constructed (GObject *object)
                             G_CALLBACK (photos_preview_nav_buttons_motion_notify),
                             self);
 
+  self->long_press_gesture = gtk_gesture_long_press_new (self->preview_view);
+  gtk_gesture_single_set_touch_only (GTK_GESTURE_SINGLE (self->long_press_gesture), TRUE);
+  g_signal_connect_swapped (self->long_press_gesture,
+                            "pressed",
+                            G_CALLBACK (photos_preview_nav_buttons_long_press_pressed),
+                            self);
+
   self->tap_gesture = gtk_gesture_multi_press_new (self->preview_view);
   gtk_gesture_single_set_touch_only (GTK_GESTURE_SINGLE (self->tap_gesture), TRUE);
+  gtk_gesture_group (self->long_press_gesture, self->tap_gesture);
   g_signal_connect_swapped (self->tap_gesture,
                             "released",
                             G_CALLBACK (photos_preview_nav_buttons_multi_press_released),
                             self);
-  g_signal_connect_swapped (self->tap_gesture,
-                            "stopped",
-                            G_CALLBACK (photos_preview_nav_buttons_multi_press_stopped),
-                            self);
 
   /* We will not need them any more */
   self->overlay = NULL;