Skip to content

Commit

Permalink
Merge pull request #5616 from nextcloud-libraries/backport/5613/next
Browse files Browse the repository at this point in the history
[next] fix(NcAppSidebar): remove hidden navigation toggle from focus trap
  • Loading branch information
ShGKme authored May 19, 2024
2 parents 6cd64b3 + 9453cbf commit d71d620
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions src/components/NcAppSidebar/NcAppSidebar.vue
Original file line number Diff line number Diff line change
Expand Up @@ -906,9 +906,6 @@ export default {
this.$refs.sidebar,
// Nextcloud Server header navigarion
document.querySelector('#header'),
// The app navigation toggle. Navigation can be opened above the sidebar
// Take the parent element, because the focus-trap requires a container with elements, not the element itself
document.querySelector('[aria-controls="app-navigation-vue"]')?.parentElement,
], {
allowOutsideClick: true,
fallbackFocus: this.$refs.closeButton,
Expand Down

0 comments on commit d71d620

Please sign in to comment.