Merge pull request #3733 from nextcloud/fix/header-menu-focus-throw

Fix throw when header menu contains no tabbable nodes
nextcloud-libraries · Feb 8, 2023 · d53a8f8 · d53a8f8
2 parents a48e4a9 + fe07eeb
commit d53a8f8
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/src/components/NcHeaderMenu/NcHeaderMenu.vue b/src/components/NcHeaderMenu/NcHeaderMenu.vue
@@ -69,7 +69,8 @@ export default {
 		:class="{ 'header-menu--opened': opened }"
 		class="header-menu">
 		<!-- Open trigger icon -->
-		<a class="header-menu__trigger"
+		<a ref="trigger"
+			class="header-menu__trigger"
 			href="#"
 			:aria-label="ariaLabel"
 			:aria-controls="`header-menu-${id}`"
@@ -254,6 +255,7 @@ export default {
 			this.focusTrap = createFocusTrap(contentContainer, {
 				allowOutsideClick: true,
 				trapStack: getTrapStack(),
+				fallbackFocus: this.$refs.trigger,
 			})
 			this.focusTrap.activate()
 		},