Skip to content

Commit

Permalink
fix(NcAppNavigationItem): Fix popper boundary element
Browse files Browse the repository at this point in the history
Signed-off-by: Christoph Wurst <[email protected]>
  • Loading branch information
ChristophWurst committed Jul 8, 2024
1 parent be3e214 commit a30de5b
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions src/components/NcAppNavigationItem/NcAppNavigationItem.vue
Original file line number Diff line number Diff line change
Expand Up @@ -609,6 +609,10 @@ export default {
}
},
mounted() {
this.actionsBoundariesElement = document.querySelector('#content-vue') || undefined
},
data() {

Check warning on line 616 in src/components/NcAppNavigationItem/NcAppNavigationItem.vue

View workflow job for this annotation

GitHub Actions / NPM lint

The "data" property should be above the "mounted" property on line 612
return {
editingValue: '',
Expand All @@ -621,6 +625,7 @@ export default {
menuOpenLocalValue: false,
focused: false,
collapsible: false,
actionsBoundariesElement: undefined,
}
},
Expand Down Expand Up @@ -653,9 +658,6 @@ export default {
undoButtonAriaLabel() {
return t('Undo changes')
},
actionsBoundariesElement() {
return document.querySelector('#content-vue') || undefined
},
},
watch: {

Check warning on line 663 in src/components/NcAppNavigationItem/NcAppNavigationItem.vue

View workflow job for this annotation

GitHub Actions / NPM lint

The "watch" property should be above the "mounted" property on line 612
Expand Down

0 comments on commit a30de5b

Please sign in to comment.