Add accel to open files via Ctrl+O (#820)

* Add accel to open files via ctrl+O

* action-open as an action of MainWindow

* Do not keep track of the SimpleAction. Let it mind its business.

* Oops

---------

Co-authored-by: Ryo Nakano <[email protected]>

Author: teamcons

src/MainWindow.vala

@@ -4,6 +4,9 @@
  */
 
 public class Music.MainWindow : Gtk.ApplicationWindow {
+    private const string ACTION_PREFIX = "win.";
+    private const string ACTION_OPEN = "action-open";
+
     private Granite.Placeholder queue_placeholder;
     private Gtk.Button repeat_button;
     private Gtk.Button shuffle_button;
@@ -78,8 +81,12 @@ public class Music.MainWindow : Gtk.ApplicationWindow {
         add_button_box.append (new Gtk.Image.from_icon_name ("document-open-symbolic"));
         add_button_box.append (add_button_label);
 
+        var open_action = new SimpleAction (ACTION_OPEN, null);
+        open_action.activate.connect (open_files);
+
         var add_button = new Gtk.Button () {
             child = add_button_box,
+            action_name = ACTION_PREFIX + ACTION_OPEN
         };
         add_button.add_css_class (Granite.STYLE_CLASS_FLAT);
 
@@ -176,6 +183,11 @@ public class Music.MainWindow : Gtk.ApplicationWindow {
 
         update_repeat_button ();
 
+        unowned var app = ((Gtk.Application) GLib.Application.get_default ());
+        app.set_accels_for_action (ACTION_PREFIX + ACTION_OPEN, {"<Ctrl>O"});
+
+        add_action (open_action);
+
         drop_target.drop.connect ((target, value, x, y) => {
             if (value.type () == typeof (Gdk.FileList)) {
                 var list = (Gdk.FileList)value;
@@ -202,8 +214,6 @@ public class Music.MainWindow : Gtk.ApplicationWindow {
             error_toast.send_notification ();
         });
 
-        add_button.clicked.connect (action_open);
-
         repeat_button.clicked.connect (() => {
             var enum_step = settings.get_enum ("repeat-mode");
             if (enum_step < 2) {
@@ -251,7 +261,7 @@ public class Music.MainWindow : Gtk.ApplicationWindow {
         }
     }
 
-    private void action_open () {
+    private void open_files () {
         var all_files_filter = new Gtk.FileFilter () {
             name = _("All files"),
         };