[gtk/wip/ebassi/shortcut: 189/267] shortcut: Add action activation

[Emmanuele Bassi <ebassi src gnome org>]

commit 987b33ec04658203b3dfc9cc5072da994e5785df
Author: Emmanuele Bassi <ebassi gnome org>
Date:   Wed Feb 5 17:08:14 2020 +0000

    shortcut: Add action activation

 gtk/gtkshortcut.c | 77 +++++++++++++++++++++++++++++++++++++++++++++++++++++++
 gtk/gtkshortcut.h |  5 ++++
 2 files changed, 82 insertions(+)
---
diff --git a/gtk/gtkshortcut.c b/gtk/gtkshortcut.c
index 69e087603b..ddf79c67b2 100644
--- a/gtk/gtkshortcut.c
+++ b/gtk/gtkshortcut.c
@@ -56,6 +56,7 @@ struct _GtkShortcut
 
   GtkShortcutTrigger *trigger;
   char *signal;
+  char *action;
   GtkShortcutFunc callback;
   gpointer user_data;
   GDestroyNotify destroy_notify;
@@ -68,6 +69,7 @@ enum
   PROP_ARGUMENTS,
   PROP_CALLBACK,
   PROP_SIGNAL,
+  PROP_ACTION,
   PROP_TRIGGER,
 
   N_PROPS
@@ -84,6 +86,7 @@ gtk_shortcut_dispose (GObject *object)
 
   g_clear_pointer (&self->trigger, gtk_shortcut_trigger_unref);
   g_clear_pointer (&self->signal, g_free);
+  g_clear_pointer (&self->action, g_free);
   g_clear_pointer (&self->args, g_variant_unref);
   if (self->callback)
     {
@@ -120,6 +123,10 @@ gtk_shortcut_get_property (GObject    *object,
       g_value_set_string (value, self->signal);
       break;
 
+    case PROP_ACTION:
+      g_value_set_string (value, self->action);
+      break;
+
     case PROP_TRIGGER:
       g_value_set_boxed (value, self->trigger);
       break;
@@ -148,6 +155,10 @@ gtk_shortcut_set_property (GObject      *object,
       gtk_shortcut_set_signal (self, g_value_get_string (value));
       break;
 
+    case PROP_ACTION:
+      gtk_shortcut_set_action (self, g_value_get_string (value));
+      break;
+
     case PROP_TRIGGER:
       gtk_shortcut_set_trigger (self, g_value_dup_boxed (value));
       break;
@@ -203,6 +214,17 @@ gtk_shortcut_class_init (GtkShortcutClass *klass)
                          P_("The action signal to emit"),
                          NULL,
                          G_PARAM_READWRITE | G_PARAM_EXPLICIT_NOTIFY | G_PARAM_STATIC_STRINGS);
+  /**
+   * GtkShortcut:action:
+   *
+   * The name of the action to activate on the widget upon activation.
+   */
+  properties[PROP_ACTION] =
+    g_param_spec_string ("action",
+                         P_("Action"),
+                         P_("The action to activate"),
+                         NULL,
+                         G_PARAM_READWRITE | G_PARAM_EXPLICIT_NOTIFY | G_PARAM_STATIC_STRINGS);
 
   /**
    * GtkShortcut:trigger:
@@ -275,6 +297,28 @@ gtk_shortcut_activate (GtkShortcut *self,
           return FALSE;
         }
 
+      return handled;
+    }
+  else if (self->action)
+    {
+      GError *error = NULL;
+      gboolean handled;
+
+      if (!gtk_binding_activate_action (G_OBJECT (widget),
+                                        self->action,
+                                        self->args,
+                                        &handled,
+                                        &error))
+        {
+          char *accelerator = gtk_shortcut_trigger_to_string (self->trigger);
+          g_warning ("gtk_shortcut_activate(): \":%s\": %s",
+                     accelerator,
+                     error->message);
+          g_clear_error (&error);
+          g_free (accelerator);
+          return FALSE;
+        }
+
       return handled;
     }
   else
@@ -362,6 +406,12 @@ gtk_shortcut_clear_activation (GtkShortcut *self)
       g_object_notify_by_pspec (G_OBJECT (self), properties[PROP_SIGNAL]);
     }
 
+  if (self->action)
+    {
+      g_clear_pointer (&self->action, g_free);
+      g_object_notify_by_pspec (G_OBJECT (self), properties[PROP_ACTION]);
+    }
+
   if (self->callback)
     {
       if (self->destroy_notify)
@@ -402,6 +452,33 @@ gtk_shortcut_set_signal (GtkShortcut *self,
   g_object_thaw_notify (G_OBJECT (self));
 }
 
+gboolean
+gtk_shortcut_has_action (GtkShortcut *self)
+{
+  g_return_val_if_fail (GTK_IS_SHORTCUT (self), FALSE);
+
+  return self->action != NULL;
+}
+
+void
+gtk_shortcut_set_action (GtkShortcut *self,
+                         const char  *action)
+{
+  g_return_if_fail (GTK_IS_SHORTCUT (self));
+
+  if (g_strcmp0 (self->action, action) == 0)
+    return;
+
+  g_object_freeze_notify (G_OBJECT (self));
+
+  gtk_shortcut_clear_activation (self);
+  self->action = g_strdup (action);
+
+  g_object_notify_by_pspec (G_OBJECT (self), properties[PROP_ACTION]);
+
+  g_object_thaw_notify (G_OBJECT (self));
+}
+
 gboolean
 gtk_shortcut_has_callback (GtkShortcut *self)
 {
diff --git a/gtk/gtkshortcut.h b/gtk/gtkshortcut.h
index 8df123597d..fda311a7c8 100644
--- a/gtk/gtkshortcut.h
+++ b/gtk/gtkshortcut.h
@@ -67,6 +67,11 @@ void            gtk_shortcut_set_callback                       (GtkShortcut
                                                                  GtkShortcutFunc         callback,
                                                                  gpointer                data,
                                                                  GDestroyNotify          destroy);
+GDK_AVAILABLE_IN_ALL
+gboolean        gtk_shortcut_has_action                         (GtkShortcut            *self);
+GDK_AVAILABLE_IN_ALL
+void            gtk_shortcut_set_action                         (GtkShortcut            *self,
+                                                                 const char             *action);
 
 G_END_DECLS