Skip to content

Commit

Permalink
fix: don't hide content from accessibility with permanent drawer
Browse files Browse the repository at this point in the history
  • Loading branch information
satya164 committed Apr 8, 2020
1 parent c4acdaa commit cb2f157
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions packages/drawer/src/views/Drawer.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -623,9 +623,11 @@ export default class DrawerView extends React.Component<Props> {
]}
>
<View
accessibilityElementsHidden={isOpen}
accessibilityElementsHidden={isOpen && drawerType !== 'permanent'}
importantForAccessibility={
isOpen ? 'no-hide-descendants' : 'auto'
isOpen && drawerType !== 'permanent'
? 'no-hide-descendants'
: 'auto'
}
style={styles.content}
>
Expand Down Expand Up @@ -656,7 +658,7 @@ export default class DrawerView extends React.Component<Props> {
/>
)}
<Animated.View
accessibilityViewIsModal={isOpen}
accessibilityViewIsModal={isOpen && drawerType !== 'permanent'}
removeClippedSubviews={Platform.OS !== 'ios'}
onLayout={this.handleDrawerLayout}
style={[
Expand Down

0 comments on commit cb2f157

Please sign in to comment.