fix: appBar Accessibility Focus (#1544)

callstack · Jan 24, 2020 · 2826d83 · 2826d83
1 parent dc7b47e
commit 2826d83
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 0 deletions.
diff --git a/src/components/Appbar/AppbarContent.tsx b/src/components/Appbar/AppbarContent.tsx
@@ -30,6 +30,10 @@ type Props = $RemoveChildren<typeof View> & {
    * Style for the title.
    */
   titleStyle?: StyleProp<TextStyle>;
+  /**
+   * Reference for the title.
+   */
+  titleRef?: React.RefObject<Text>;
   /**
    * Text for the subtitle.
    */
@@ -62,6 +66,7 @@ class AppbarContent extends React.Component<Props> {
       subtitleStyle,
       onPress,
       style,
+      titleRef,
       titleStyle,
       theme,
       title,
@@ -78,6 +83,7 @@ class AppbarContent extends React.Component<Props> {
       <TouchableWithoutFeedback onPress={onPress}>
         <View style={[styles.container, style]} {...rest}>
           <Text
+            ref={titleRef}
             style={[
               {
                 color: titleColor,
@@ -87,6 +93,7 @@ class AppbarContent extends React.Component<Props> {
               titleStyle,
             ]}
             numberOfLines={1}
+            accessible
             accessibilityTraits="header"
             // @ts-ignore
             accessibilityRole={Platform.OS === 'web' ? 'heading' : 'header'}

diff --git a/src/components/__tests__/Appbar/__snapshots__/Appbar.test.js.snap b/src/components/__tests__/Appbar/__snapshots__/Appbar.test.js.snap
@@ -417,6 +417,7 @@ exports[`Appbar passes additional props to AppbarBackAction, AppbarContent and A
     <Text
       accessibilityRole="header"
       accessibilityTraits="header"
+      accessible={true}
       numberOfLines={1}
       style={
         Array [