Merge lp:~mank319/pantheon-files/navigation-buttons into lp:~elementary-apps/pantheon-files/trunk

[Manuel Kehl]

XF86Back and XF86Forward can now be used to navigate back and forth in pantheon-files, which probably is what most users with such buttons would expect.

XF86Back and XF86Forward can now be used to navigate back and forth in pantheon-files, which probably is what most users with such buttons would expect.

I adapted the mechanism of "go_up" to make sure that I meet the expectations of your code base. I hope the changes are useful and would love to see them merged!

Please tell me if there is anything about my contribution, that you would like me to change, because both Vala and Launchpad/Bazaar are completely new to me ;-)

[lp:~mank319/pantheon-files/navigation-buttons]

1630. By Manuel Kehl on 2014-11-02

Bound the signals to the corresponding keys (XF86Back, XF86Forward).

[Jeremy Wootten (jeremywootten)]

The branch works as expected and the code is clean.

Thanks for your contribution Manuel!

review: Approve

[Manuel Kehl (mank319)]

Thank you for the review!

On So, Nov 2, 2014 at 12:05 , Jeremy Wootten <email address hidden>
wrote:
> Review: Approve
>
> The branch works as expected and the code is clean.
>
> Thanks for your contribution Manuel!
> --
> https://code.launchpad.net/~mank319/pantheon-files/navigation-buttons/+merge/239882
> You are the owner of lp:~mank319/pantheon-files/navigation-buttons.

[Jeremy Wootten (jeremywootten)]

No problem, I hope you are able to contribute again in the future - there
is plenty more work to do on Files!

On 2 November 2014 11:48, Manuel Kehl <email address hidden> wrote:

> Thank you for the review!
>
> On So, Nov 2, 2014 at 12:05 , Jeremy Wootten <email address hidden>
> wrote:
> > Review: Approve
> >
> > The branch works as expected and the code is clean.
> >
> > Thanks for your contribution Manuel!
> > --
> >
> https://code.launchpad.net/~mank319/pantheon-files/navigation-buttons/+merge/239882
> > You are the owner of lp:~mank319/pantheon-files/navigation-buttons.
>
> --
>
> https://code.launchpad.net/~mank319/pantheon-files/navigation-buttons/+merge/239882
> You are reviewing the proposed merge of
> lp:~mank319/pantheon-files/navigation-buttons into lp:pantheon-files.
>

=== modified file 'src/View/Window.vala'
--- src/View/Window.vala	2014-10-15 12:26:22 +0000
+++ src/View/Window.vala	2014-11-02 10:09:28 +0000
@@ -285,6 +285,8 @@
             if (is_first_window) {
                 unowned Gtk.BindingSet binding_set = Gtk.BindingSet.by_class (get_class ());
                 Gtk.BindingEntry.add_signal (binding_set, Gdk.keyval_from_name ("BackSpace"), 0, "go_up", 0);
+                Gtk.BindingEntry.add_signal (binding_set, Gdk.keyval_from_name ("XF86Back"), 0, "go_back", 0);
+                Gtk.BindingEntry.add_signal (binding_set, Gdk.keyval_from_name ("XF86Forward"), 0, "go_forward", 0);
                 Gtk.BindingEntry.add_signal (binding_set, Gdk.keyval_from_name ("L"), Gdk.ModifierType.CONTROL_MASK, "edit_path", 0);
             }
 
@@ -306,6 +308,16 @@
         public virtual signal void go_up () {
             action_go_up ();
         }
+        
+        [Signal (action=true)]
+        public virtual signal void go_back () {
+            action_go_back ();
+        }
+        
+        [Signal (action=true)]
+        public virtual signal void go_forward () {
+            action_go_forward ();
+        }
 
         [Signal (action=true)]
         public virtual signal void edit_path () {