Skip to content
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

Closed
2 tasks done
AndyScherzinger opened this issue Jul 20, 2022 · 8 comments · Fixed by #2762
Assignees
Labels

Comments

@AndyScherzinger
Copy link
Member

AndyScherzinger commented Jul 20, 2022

Moved over from nextcloud/server#21443

Details see wcagFixes.xlsx as well as discussions in nextcloud/server#21443 (comment)

c&op from the content:

4 Keyboard Currently keyboard accessible using keyboard shortcuts common in text editors, however there is a WAI-ARIA best practice None 2.1.1 - Text Editor Text Editor is not configurable due to it being another third party application

Is the missing role="menubar" on formatting bar as well as the missing role="menuitem" on each of the items what you are referring to? That we should of course fix.

Ref @juliushaertl for the Text editor:

ToDo

@AndyScherzinger
Copy link
Member Author

@juliushaertl assigned you just as a "ping"

@juliushaertl
Copy link
Member

role menubar and menuitem were already addressed in #2715

@juliushaertl
Copy link
Member

Currently keyboard accessible using keyboard shortcuts common in text editors, however there is a WAI-ARIA best practice

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:

Text Editor is not configurable due to it being another third party application

What should be configurable here? Is there a way we can clarify that?

@AndyScherzinger
Copy link
Member Author

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.

@jancborchardt
Copy link
Member

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?

@juliushaertl
Copy link
Member

Yes, we have pretty standard shortcuts there all involving some combination of Ctrl / Ctrl+Shift 👍

@AndyScherzinger
Copy link
Member Author

@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.

@juliushaertl
Copy link
Member

Yes, added as a todo item to the first post.

@juliushaertl juliushaertl removed their assignment Jul 21, 2022
@vinicius73 vinicius73 self-assigned this Jul 25, 2022
@juliushaertl juliushaertl added bug Something isn't working 1. to develop and removed bug Something isn't working labels Jul 26, 2022
vinicius73 pushed a commit that referenced this issue Jul 29, 2022
Signed-off-by: Vinicius Reis <vinicius.reis@nextcloud.com>
vinicius73 pushed a commit that referenced this issue Jul 29, 2022
- strikethrough
- blockquote
Signed-off-by: Vinicius Reis <vinicius.reis@nextcloud.com>
vinicius73 pushed a commit that referenced this issue Jul 29, 2022
Signed-off-by: Vinicius Reis <vinicius.reis@nextcloud.com>
vinicius73 pushed a commit that referenced this issue Jul 29, 2022
Signed-off-by: Vinicius Reis <vinicius.reis@nextcloud.com>
vinicius73 pushed a commit that referenced this issue Jul 29, 2022
Signed-off-by: Vinicius Reis <vinicius.reis@nextcloud.com>
juliushaertl pushed a commit that referenced this issue Jul 30, 2022
Signed-off-by: Vinicius Reis <vinicius.reis@nextcloud.com>
juliushaertl pushed a commit that referenced this issue Jul 30, 2022
- strikethrough
- blockquote
Signed-off-by: Vinicius Reis <vinicius.reis@nextcloud.com>
juliushaertl pushed a commit that referenced this issue Jul 30, 2022
Signed-off-by: Vinicius Reis <vinicius.reis@nextcloud.com>
juliushaertl pushed a commit that referenced this issue Jul 30, 2022
Signed-off-by: Vinicius Reis <vinicius.reis@nextcloud.com>
juliushaertl pushed a commit that referenced this issue Jul 30, 2022
Signed-off-by: Vinicius Reis <vinicius.reis@nextcloud.com>
nextcloud-command pushed a commit that referenced this issue Aug 1, 2022
Signed-off-by: Vinicius Reis <vinicius.reis@nextcloud.com>
nextcloud-command pushed a commit that referenced this issue Aug 1, 2022
- strikethrough
- blockquote
Signed-off-by: Vinicius Reis <vinicius.reis@nextcloud.com>
nextcloud-command pushed a commit that referenced this issue Aug 1, 2022
Signed-off-by: Vinicius Reis <vinicius.reis@nextcloud.com>
nextcloud-command pushed a commit that referenced this issue Aug 1, 2022
Signed-off-by: Vinicius Reis <vinicius.reis@nextcloud.com>
nextcloud-command pushed a commit that referenced this issue Aug 1, 2022
Signed-off-by: Vinicius Reis <vinicius.reis@nextcloud.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants