Merge pull request #6023 from nextcloud-libraries/fix/menu-role

[next] fix(NcActions): Role `menu` needs a label assigned so label by trigger
This commit is contained in:
Ferdinand Thiessen 2024-08-29 18:31:12 +02:00 коммит произвёл GitHub
Родитель b8b26d2b86 5cf16e52a8
Коммит b19cc1dfc8
Не найден ключ, соответствующий данной подписи
Идентификатор ключа GPG: B5690EEEBB952194
1 изменённых файлов: 3 добавлений и 2 удалений

Просмотреть файл

@ -1779,8 +1779,9 @@ export default {
tabindex: '-1',
ref: 'menuList',
role: this.config.popupRole,
// Dialog must have a label
'aria-labelledby': this.actionsMenuSemanticType === 'dialog' ? triggerRandomId : undefined,
// For most roles a label is required (dialog, menu), but also in general nothing speaks against labelling a list.
// It is even recommended to do so.
'aria-labelledby': triggerRandomId,
'aria-modal': this.actionsMenuSemanticType === 'dialog' ? 'true' : undefined,
}, [
actions,