From 395e53af736dfbf0dbf0bcaab94d9abf5ea3e94a Mon Sep 17 00:00:00 2001
From: Brent Vatne <brentvatne@gmail.com>
Date: Fri, 12 Oct 2018 12:39:44 -0700
Subject: [PATCH] Fix modular header back button to default to showing Back and
 respect visibility prop

---
 packages/stack/example/src/SimpleStack.js     |  9 ++
 packages/stack/src/views/Header/Header.js     |  5 +-
 .../views/Header/ModularHeaderBackButton.js   | 83 +++++++++++--------
 3 files changed, 62 insertions(+), 35 deletions(-)

diff --git a/packages/stack/example/src/SimpleStack.js b/packages/stack/example/src/SimpleStack.js
index 68b40f00..1f10eea0 100644
--- a/packages/stack/example/src/SimpleStack.js
+++ b/packages/stack/example/src/SimpleStack.js
@@ -3,6 +3,10 @@ import { Button, View, Text } from 'react-native';
 import { createStackNavigator } from 'react-navigation-stack';
 
 class ListScreen extends React.Component {
+  static navigationOptions = {
+    title: 'List',
+  };
+
   render() {
     return (
       <View
@@ -29,6 +33,10 @@ class ListScreen extends React.Component {
 }
 
 class DetailsScreen extends React.Component {
+  static navigationOptions = {
+    title: 'Details',
+  };
+
   render() {
     return (
       <View
@@ -72,5 +80,6 @@ export default createStackNavigator(
     // these are the defaults
     cardShadowEnabled: true,
     cardOverlayEnabled: false,
+    // headerTransitionPreset: 'uikit',
   }
 );
diff --git a/packages/stack/src/views/Header/Header.js b/packages/stack/src/views/Header/Header.js
index 31a18f97..b84b69bf 100644
--- a/packages/stack/src/views/Header/Header.js
+++ b/packages/stack/src/views/Header/Header.js
@@ -247,6 +247,7 @@ class Header extends React.PureComponent {
         onPress={goBack}
         ButtonContainerComponent={ButtonContainerComponent}
         LabelContainerComponent={LabelContainerComponent}
+        backTitleVisible={this.props.backTitleVisible}
         pressColorAndroid={options.headerPressColorAndroid}
         tintColor={options.headerTintColor}
         backImage={options.headerBackImage}
@@ -684,13 +685,13 @@ const styles = StyleSheet.create({
   iconMaskFillerRect: {
     flex: 1,
     backgroundColor: '#d8d8d8',
-    marginLeft: -3,
+    marginLeft: -5,
   },
   iconMask: {
     // These are mostly the same as the icon in ModularHeaderBackButton
     height: 21,
     width: 12,
-    marginLeft: 9,
+    marginLeft: 8.5,
     marginTop: -0.5, // resizes down to 20.5
     alignSelf: 'center',
     resizeMode: 'contain',
diff --git a/packages/stack/src/views/Header/ModularHeaderBackButton.js b/packages/stack/src/views/Header/ModularHeaderBackButton.js
index aa0f2f3c..b3ef0851 100644
--- a/packages/stack/src/views/Header/ModularHeaderBackButton.js
+++ b/packages/stack/src/views/Header/ModularHeaderBackButton.js
@@ -23,7 +23,7 @@ class ModularHeaderBackButton extends React.PureComponent {
   };
 
   _renderBackImage() {
-    const { backImage, title, tintColor } = this.props;
+    const { backImage, backTitleVisible, tintColor } = this.props;
 
     let BackImage;
     let props;
@@ -34,14 +34,13 @@ class ModularHeaderBackButton extends React.PureComponent {
       BackImage = backImage;
       props = {
         tintColor,
-        title,
       };
     } else {
       BackImage = Image;
       props = {
         style: [
           styles.icon,
-          !!title && styles.iconWithTitle,
+          !!backTitleVisible && styles.iconWithTitle,
           !!tintColor && { tintColor },
         ],
         source: defaultBackImage,
@@ -51,6 +50,50 @@ class ModularHeaderBackButton extends React.PureComponent {
     return <BackImage {...props} />;
   }
 
+  _getTitleText = () => {
+    const { width, title, truncatedTitle } = this.props;
+
+    let { initialTextWidth } = this.state;
+
+    if (title === null) {
+      return null;
+    } else if (!title) {
+      return truncatedTitle;
+    } else if (initialTextWidth && width && initialTextWidth > width) {
+      return truncatedTitle;
+    } else {
+      return title.length > 8 ? truncatedTitle : title;
+    }
+  };
+
+  _maybeRenderTitle() {
+    const { backTitleVisible, titleStyle, tintColor } = this.props;
+    let backTitleText = this._getTitleText();
+
+    if (!backTitleVisible || backTitleText === null) {
+      return null;
+    }
+
+    const { LabelContainerComponent } = this.props;
+
+    return (
+      <LabelContainerComponent>
+        <Text
+          accessible={false}
+          onLayout={this._onTextLayout}
+          style={[
+            styles.title,
+            !!tintColor && { color: tintColor },
+            titleStyle,
+          ]}
+          numberOfLines={1}
+        >
+          {this._getTitleText()}
+        </Text>
+      </LabelContainerComponent>
+    );
+  }
+
   render() {
     const {
       onPress,
@@ -61,25 +104,12 @@ class ModularHeaderBackButton extends React.PureComponent {
       truncatedTitle,
     } = this.props;
 
-    const renderTruncated =
-      this.state.initialTextWidth && width
-        ? this.state.initialTextWidth > width
-        : false;
-
-    let backButtonTitle = renderTruncated ? truncatedTitle : title;
-
-    // TODO: When we've sorted out measuring in the header, let's revisit this.
-    // This is clearly a bad workaround.
-    if (backButtonTitle && backButtonTitle.length > 8) {
-      backButtonTitle = truncatedTitle;
-    }
-
-    const { ButtonContainerComponent, LabelContainerComponent } = this.props;
+    const { ButtonContainerComponent } = this.props;
 
     return (
       <TouchableItem
         accessibilityComponentType="button"
-        accessibilityLabel={backButtonTitle}
+        accessibilityLabel={title ? `${title}, back` : 'Go back'}
         accessibilityTraits="button"
         testID="header-back"
         delayPressIn={0}
@@ -91,21 +121,7 @@ class ModularHeaderBackButton extends React.PureComponent {
           <ButtonContainerComponent>
             {this._renderBackImage()}
           </ButtonContainerComponent>
-          {typeof backButtonTitle === 'string' && (
-            <LabelContainerComponent>
-              <Text
-                onLayout={this._onTextLayout}
-                style={[
-                  styles.title,
-                  !!tintColor && { color: tintColor },
-                  titleStyle,
-                ]}
-                numberOfLines={1}
-              >
-                {backButtonTitle}
-              </Text>
-            </LabelContainerComponent>
-          )}
+          {this._maybeRenderTitle()}
         </View>
       </TouchableItem>
     );
@@ -117,6 +133,7 @@ const styles = StyleSheet.create({
     alignItems: 'center',
     flexDirection: 'row',
     backgroundColor: 'transparent',
+    marginBottom: 1,
   },
   title: {
     fontSize: 17,