-
Notifications
You must be signed in to change notification settings - Fork 94
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[BITV/A11Y] Currently keyboard accessible using keyboard shortcuts common in text editors, however there is a WAI-ARIA best practice #2735
Comments
@juliushaertl assigned you just as a "ping" |
role menubar and menuitem were already addressed in #2715 |
As far as I get that this is about adding keyboard shortcuts through the aria-key shortcuts attribute: https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/Attributes/aria-keyshortcuts About the other topics I'm not sure I understand the report:
What should be configurable here? Is there a way we can clarify that? |
don't know for sure, best to ask @jancborchardt - I would guess it is about configuring the shortcut them self while that might be an obsolete info and Jan might have a updated view on that, maybe. |
Tbh I also don't understand their remark @juliushaertl quoted. Normally standard keyboard shortcuts should be ok as long as they are not single letters which might interfere with typing. And we don't use any of the like in Text, instead having standard shortcuts like Ctrl+B for bold, so that seems fine? |
Yes, we have pretty standard shortcuts there all involving some combination of Ctrl / Ctrl+Shift 👍 |
@juliushaertl so did you also do?: Expose shortcuts through aria-keyshortcuts - if no then I think this is the only remaining issue looking at the ARIA spec. |
Yes, added as a todo item to the first post. |
Signed-off-by: Vinicius Reis <vinicius.reis@nextcloud.com>
- strikethrough - blockquote Signed-off-by: Vinicius Reis <vinicius.reis@nextcloud.com>
Signed-off-by: Vinicius Reis <vinicius.reis@nextcloud.com>
Signed-off-by: Vinicius Reis <vinicius.reis@nextcloud.com>
Signed-off-by: Vinicius Reis <vinicius.reis@nextcloud.com>
Signed-off-by: Vinicius Reis <vinicius.reis@nextcloud.com>
- strikethrough - blockquote Signed-off-by: Vinicius Reis <vinicius.reis@nextcloud.com>
Signed-off-by: Vinicius Reis <vinicius.reis@nextcloud.com>
Signed-off-by: Vinicius Reis <vinicius.reis@nextcloud.com>
Signed-off-by: Vinicius Reis <vinicius.reis@nextcloud.com>
Signed-off-by: Vinicius Reis <vinicius.reis@nextcloud.com>
- strikethrough - blockquote Signed-off-by: Vinicius Reis <vinicius.reis@nextcloud.com>
Signed-off-by: Vinicius Reis <vinicius.reis@nextcloud.com>
Signed-off-by: Vinicius Reis <vinicius.reis@nextcloud.com>
Signed-off-by: Vinicius Reis <vinicius.reis@nextcloud.com>
Moved over from nextcloud/server#21443
Details see wcagFixes.xlsx as well as discussions in nextcloud/server#21443 (comment)
c&op from the content:
Is the missing
role="menubar"
on formatting bar as well as the missingrole="menuitem"
on each of the items what you are referring to? That we should of course fix.Ref @juliushaertl for the Text editor:
ToDo
The text was updated successfully, but these errors were encountered: