fix(NcActions): move dialog a11y attrs from UL to container

Signed-off-by: Grigorii K. Shartsev <me@shgk.me>
nextcloud-libraries · Mar 8, 2024 · 5b35b7e · 5b35b7e
1 parent eb173f2
commit 5b35b7e
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/src/components/NcActions/NcActions.vue b/src/components/NcActions/NcActions.vue
@@ -1192,6 +1192,7 @@ export default {
 					},
 					popoverContainerA11yAttrs: {},
 					popoverUlA11yAttrs: {
+						id: this.randomId,
 						role: 'menu',
 					},
 				},
@@ -1212,13 +1213,14 @@ export default {
 					triggerA11yAttr: {
 						'aria-controls': this.opened ? this.randomId : null,
 					},
-					popoverContainerA11yAttrs: {},
-					popoverUlA11yAttrs: {
+					popoverContainerA11yAttrs: {
+						id: this.randomId,
 						role: 'dialog',
 						// Dialog must have a label
 						'aria-labelledby': this.triggerRandomId,
 						'aria-modal': 'true',
 					},
+					popoverUlA11yAttrs: {},
 				},
 				tooltip: {
 					popupRole: undefined,
@@ -1836,7 +1838,6 @@ export default {
 					}, [
 						h('ul', {
 							attrs: {
-								id: this.randomId,
 								tabindex: '-1',
 								...this.config.popoverUlA11yAttrs,
 							},