You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I was testing screen readers with the react-native-keyboard-controller changes to ensure it still worked, and realized v10 itself has some accessibility issues we need to fix.
A lot of this comes from the swap from DrawerLayout from react-native-gesture-handler to Drawer from react-navigation in #2528. We shouldn't swap back to DrawerLayout: according to software-mansion/react-native-gesture-handler#2414, this component hasn't been maintained and will likely continue to have issues due to the use of setNativeProps.
This means we'll need to either patch Drawer or find/build a replacement. I noticed the Drawer component is either at v4 or v7, and we're using v4, which makes me wonder if there might be updates for v7 that would be relevant to us
If building a replacement, DrawerLayout will still be a useful reference for which accessibility traits need managing, even if they do need to be set differently.
Ideally, we could also add tests to ensure the accessibility traits for the drawer are set correctly in the future.
List of issues I found follows, may not be exhaustive:
iOS
Demo showroom issues
voiceover linear navigation goes through menu items when drawer isn’t open
once menu is open, can’t focus menu items without directly touching them, gets stuck on back button
can’t actually focus content on demoshowroom screen at all
Navigating from Community → Podcast does not appropriately refocus cursor at top of screen
podcast screen
double-tap and hold to favorite episode didn’t work
not all text content is read out on podcast cards
Android
Says menu icon on drawer is unlabeled
menu back button not focusable at all
date and time labels on podcasts are not read out
double-tap and hold does work here!
iOSDrawerFromReactNavigation.MP4
Ignite version
10
Additional info
n/a
The text was updated successfully, but these errors were encountered:
Describe the bug
I was testing screen readers with the react-native-keyboard-controller changes to ensure it still worked, and realized v10 itself has some accessibility issues we need to fix.
A lot of this comes from the swap from
DrawerLayout
from react-native-gesture-handler toDrawer
from react-navigation in #2528. We shouldn't swap back toDrawerLayout
: according to software-mansion/react-native-gesture-handler#2414, this component hasn't been maintained and will likely continue to have issues due to the use ofsetNativeProps
.This means we'll need to either patch
Drawer
or find/build a replacement. I noticed theDrawer
component is either at v4 or v7, and we're using v4, which makes me wonder if there might be updates for v7 that would be relevant to usIf building a replacement,
DrawerLayout
will still be a useful reference for which accessibility traits need managing, even if they do need to be set differently.Ideally, we could also add tests to ensure the accessibility traits for the drawer are set correctly in the future.
List of issues I found follows, may not be exhaustive:
iOS
Android
iOSDrawerFromReactNavigation.MP4
Ignite version
10
Additional info
n/a
The text was updated successfully, but these errors were encountered: