From 40ef76a6869991fd9a36156e41ddf7cfb4d39bae Mon Sep 17 00:00:00 2001
From: Daniel Odrinski <m4tr1xc0d3r@protonmail.com>
Date: Tue, 13 Jun 2023 10:37:02 +0100
Subject: [PATCH] [Peek]Add up/down arrow key item navigation (#26644)

---
 src/modules/peek/Peek.UI/MainWindow.xaml        | 6 ++++--
 src/modules/peek/Peek.UI/MainWindow.xaml.cs     | 8 ++++----
 src/modules/peek/Peek.UI/MainWindowViewModel.cs | 4 ++--
 3 files changed, 10 insertions(+), 8 deletions(-)

diff --git a/src/modules/peek/Peek.UI/MainWindow.xaml b/src/modules/peek/Peek.UI/MainWindow.xaml
index e8d8b2aab0..1b6ad103b6 100644
--- a/src/modules/peek/Peek.UI/MainWindow.xaml
+++ b/src/modules/peek/Peek.UI/MainWindow.xaml
@@ -16,8 +16,10 @@
     </Window.SystemBackdrop>
     <Grid KeyboardAcceleratorPlacementMode="Hidden">
         <Grid.KeyboardAccelerators>
-            <KeyboardAccelerator Key="Left" Invoked="LeftNavigationInvoked" />
-            <KeyboardAccelerator Key="Right" Invoked="RightNavigationInvoked" />
+            <KeyboardAccelerator Key="Left" Invoked="PreviousNavigationInvoked" />
+            <KeyboardAccelerator Key="Up" Invoked="PreviousNavigationInvoked" />
+            <KeyboardAccelerator Key="Right" Invoked="NextNavigationInvoked" />
+            <KeyboardAccelerator Key="Down" Invoked="NextNavigationInvoked" />
             <KeyboardAccelerator Key="Escape" Invoked="EscKeyInvoked" />
         </Grid.KeyboardAccelerators>
 
diff --git a/src/modules/peek/Peek.UI/MainWindow.xaml.cs b/src/modules/peek/Peek.UI/MainWindow.xaml.cs
index 146346f726..4a40a1b623 100644
--- a/src/modules/peek/Peek.UI/MainWindow.xaml.cs
+++ b/src/modules/peek/Peek.UI/MainWindow.xaml.cs
@@ -94,14 +94,14 @@ namespace Peek.UI
             }
         }
 
-        private void LeftNavigationInvoked(KeyboardAccelerator sender, KeyboardAcceleratorInvokedEventArgs args)
+        private void PreviousNavigationInvoked(KeyboardAccelerator sender, KeyboardAcceleratorInvokedEventArgs args)
         {
-            ViewModel.AttemptLeftNavigation();
+            ViewModel.AttemptPreviousNavigation();
         }
 
-        private void RightNavigationInvoked(KeyboardAccelerator sender, KeyboardAcceleratorInvokedEventArgs args)
+        private void NextNavigationInvoked(KeyboardAccelerator sender, KeyboardAcceleratorInvokedEventArgs args)
         {
-            ViewModel.AttemptRightNavigation();
+            ViewModel.AttemptNextNavigation();
         }
 
         private void EscKeyInvoked(KeyboardAccelerator sender, KeyboardAcceleratorInvokedEventArgs args)
diff --git a/src/modules/peek/Peek.UI/MainWindowViewModel.cs b/src/modules/peek/Peek.UI/MainWindowViewModel.cs
index c2084fa74e..7a01340d5e 100644
--- a/src/modules/peek/Peek.UI/MainWindowViewModel.cs
+++ b/src/modules/peek/Peek.UI/MainWindowViewModel.cs
@@ -66,7 +66,7 @@ namespace Peek.UI
             Items = null;
         }
 
-        public void AttemptLeftNavigation()
+        public void AttemptPreviousNavigation()
         {
             if (NavigationThrottleTimer.IsEnabled)
             {
@@ -80,7 +80,7 @@ namespace Peek.UI
             CurrentItem = Items?.ElementAtOrDefault(CurrentIndex);
         }
 
-        public void AttemptRightNavigation()
+        public void AttemptNextNavigation()
         {
             if (NavigationThrottleTimer.IsEnabled)
             {