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

Improve Accessibility in the Block List #14

Open
maxme opened this issue Mar 20, 2018 · 4 comments
Open

Improve Accessibility in the Block List #14

maxme opened this issue Mar 20, 2018 · 4 comments

Comments

@maxme
Copy link
Contributor

maxme commented Mar 20, 2018

Users should be able to focus each block using the keyboard (or native accessibility features).

We should probably set the accessible attribute to true on the root BlockHolder element.

More info here: https://facebook.github.io/react-native/docs/accessibility.html

@hypest hypest added the HACK week HACK week June 2020 label Jun 7, 2020
@guarani guarani self-assigned this Jun 9, 2020
@maxme maxme changed the title Improve Accessibility in the Block Manager Improve Accessibility in the Block List Jun 9, 2020
@guarani
Copy link
Contributor

guarani commented Jun 10, 2020

Hey @maxme – regarding the external keyboard aspect of this, there's an issue on Android where after moving focus away from a rich text, it's difficult to bring focus back to it to continue editing:

The ripple effect is when I'm tapping space bar on the external keyboard which should focus the rich text but doesn't

On iOS, I didn't find any issues when editing with an external keyboard (on an iPhone with Full Keyboard Access enabled in device accessibility settings).

I'm looking to see if the fix will be in RN or native, but since rich text is Aztec and this issue is not present on iOS, it may be an Aztec-Android issue.

Can I get confirmation this is the sort of issue you were referring to in this ticket?

@guarani
Copy link
Contributor

guarani commented Jun 15, 2020

The Android-specific keyboard issue mentioned above seems related to native Aztec code on Android. @hypest suggested commenting out these two sections of code to see if they're causing the issue.


and
aztecText.setFocusableInTouchMode(false);
aztecText.setFocusable(false);

I commented these out, ran yarn android, and this indeed improves things—I'm now able to move focus between rich text areas using only the keyboard.

Screen recording

I think this will need an Android developer (cc @marecar3 @cameronvoell) if we want to make this fix permanent.
Also, I noticed that when using tabs to navigate, the first tab highlights a block and the second puts the cursor on it (focuses it). When the block is highlighted (first tab press), subsequently hitting Space Bar probably should (?) allow you to use the block toolbar buttons (e.g. undo/redo/delete), but it instead sends the focus back up to the heading block, which seems incorrect (see below gif).

Screen recording

@hypest
Copy link
Contributor

hypest commented Jun 16, 2020

but it instead sends the focus back up to the heading block, which seems incorrect

I think that's the first Aztec isntance claiming focus, exactly the issue those lines I suggested you try to remove are trying to combat actually. So, I think for this quick experiment, yes, it's expected to see this side effect when commenting them out.

@hypest
Copy link
Contributor

hypest commented Oct 22, 2020

This ticket is not worked on at the moment so, I'll clear the assignee field to denote so.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants