-
Notifications
You must be signed in to change notification settings - Fork 58
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
Comments
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'tOn 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? |
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. Lines 228 to 229 in dd2a9a7
and Lines 107 to 108 in a4419c3
I commented these out, ran I think this will need an Android developer (cc @marecar3 @cameronvoell) if we want to make this fix permanent. |
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. |
This ticket is not worked on at the moment so, I'll clear the assignee field to denote so. |
Users should be able to focus each block using the keyboard (or native accessibility features).
We should probably set the
accessible
attribute totrue
on the rootBlockHolder
element.More info here: https://facebook.github.io/react-native/docs/accessibility.html
The text was updated successfully, but these errors were encountered: