From 6b3f5138566bbf9ac26c6191365986fcc879a7e1 Mon Sep 17 00:00:00 2001
From: Varun Patil <varunpatil@ucla.edu>
Date: Wed, 3 Apr 2024 18:05:14 -0700
Subject: [PATCH] fix(sidebar): propagate keydown unless mobile

Signed-off-by: Varun Patil <varunpatil@ucla.edu>
---
 src/components/NcAppSidebar/NcAppSidebar.vue | 14 +++++++++++++-
 1 file changed, 13 insertions(+), 1 deletion(-)

diff --git a/src/components/NcAppSidebar/NcAppSidebar.vue b/src/components/NcAppSidebar/NcAppSidebar.vue
index dd6c14bd36..5b83533df2 100644
--- a/src/components/NcAppSidebar/NcAppSidebar.vue
+++ b/src/components/NcAppSidebar/NcAppSidebar.vue
@@ -379,7 +379,7 @@ A working alternative would be using an icon together with an `aria-label`:
 			ref="sidebar"
 			class="app-sidebar"
 			:aria-labelledby="`app-sidebar-vue-${uid}__header`"
-			@keydown.esc.stop="isMobile && closeSidebar()">
+			@keydown.esc="onKeydownEsc">
 			<header :class="{
 					'app-sidebar-header--with-figure': hasFigure,
 					'app-sidebar-header--compact': compact,
@@ -794,6 +794,18 @@ export default {
 			}
 		},
 
+		/**
+		 * Close the sidebar on pressing the escape key on mobile
+		 *
+		 * @param {KeyboardEvent} event key down event
+		 */
+		onKeydownEsc(event) {
+			if (this.isMobile) {
+				event.stopPropagation()
+				this.closeSidebar()
+			}
+		},
+
 		onBeforeEnter(element) {
 			/**
 			 * The sidebar is opening and the transition is in progress