[BITV] Modal dialog accessibility

[PVince81]

Needs to be a proper modal, see https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/Roles/dialog_role

• [x] The div needs to have role="dialog": https://github.com/nextcloud/nextcloud-vue/issues/1590 https://github.com/nextcloud/nextcloud-vue/pull/2696
• [x] nextcloud/nextcloud-vue#2810
• [x] Keep focus within the dialog, prevent tabbing through elements in background: https://github.com/nextcloud/nextcloud-vue/issues/1591
• [x] nextcloud/nextcloud-vue#2811
• [x] Prev/next arrows are not focusable at all, and should be buttons https://github.com/nextcloud/nextcloud-vue/pull/2695
• [x] nextcloud/nextcloud-vue#2815
• [x] nextcloud/nextcloud-vue#2816
• ~~nextcloud/viewer#1311~~
• ~~nextcloud/viewer#1312~~
• [x] nextcloud/nextcloud-vue#2814
• [x] https://github.com/nextcloud/nextcloud-vue/issues/2954
• [ ] https://github.com/nextcloud/nextcloud-vue/issues/3088

[vanpertsch]

https://github.com/nextcloud/nextcloud-vue/pull/2654

[PVince81]

I've transferred the "image viewer" tickets to the viewer repo

originally that checkbox list was a dump of my findings where I was looking at various modals and their contents

[PVince81]

remaining task, was a result of an accessibility review and an answer to the question whether we should automatically focus inside a dialog that was just opened: https://github.com/nextcloud/nextcloud-vue/issues/3088

[JuliaKirschenheuter]

all closed