fix(NcActions): Role menu needs a label assigned so label by trigger

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
nextcloud-libraries · Aug 6, 2024 · 17f5bc1 · 17f5bc1
1 parent e483f7c
commit 17f5bc1
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/src/components/NcActions/NcActions.vue b/src/components/NcActions/NcActions.vue
@@ -1262,6 +1262,7 @@ export default {
 					},
 					popoverContainerA11yAttrs: {},
 					popoverUlA11yAttrs: {
+						'aria-labelledby': this.triggerRandomId,
 						id: this.randomId,
 						role: 'menu',
 					},
@@ -1311,7 +1312,11 @@ export default {
 					withFocusTrap: true,
 					triggerA11yAttr: {},
 					popoverContainerA11yAttrs: {},
-					popoverUlA11yAttrs: {},
+					popoverUlA11yAttrs: {
+						// there is nothing against labelling a list, it is mostly recommended
+						// so as we do not know the dialog type lets include the label
+						'aria-labelledby': this.triggerRandomId,
+					},
 				},
 			}
 			return configs[this.actionsMenuSemanticType]