Skip to content

Commit

Permalink
Merge pull request #4864 from nextcloud-libraries/fix/noid/lower-navi…
Browse files Browse the repository at this point in the history
…gation-toggle

fix(NcAppNavigation): make toggle button not the first element for focus-trap
  • Loading branch information
Antreesy authored Nov 22, 2023
2 parents 7e41e70 + 5c96562 commit 6e2963a
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions src/components/NcAppNavigation/NcAppNavigation.vue
Original file line number Diff line number Diff line change
Expand Up @@ -57,9 +57,6 @@ emit('toggle-navigation', {
<div ref="appNavigationContainer"
class="app-navigation"
:class="{'app-navigation--close':!open }">
<div class="app-navigation__toggle-wrapper">
<NcAppNavigationToggle :open="open" @update:open="toggleNavigation" />
</div>
<nav id="app-navigation-vue"
:aria-hidden="open ? 'false' : 'true'"
:aria-label="ariaLabel || undefined"
Expand All @@ -76,6 +73,7 @@ emit('toggle-navigation', {
<!-- Footer for e.g. AppNavigationSettings -->
<slot name="footer" />
</nav>
<NcAppNavigationToggle :open="open" @update:open="toggleNavigation" />
</div>
</template>

Expand Down

0 comments on commit 6e2963a

Please sign in to comment.