Fix network color indicator on transactionHeader for connect screens (#…

…1628) * Use network colors
MetaMask · Jun 15, 2020 · 8c039a7 · 8c039a7
1 parent 0f80b11
commit 8c039a7
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 9 deletions.
diff --git a/app/components/UI/NavbarBrowserTitle/index.js b/app/components/UI/NavbarBrowserTitle/index.js
@@ -119,7 +119,7 @@ class NavbarBrowserTitle extends PureComponent {
 					</Text>
 				</View>
 				<View style={styles.network}>
-					<View style={[styles.networkIcon, color ? { backgroundColor: color } : styles.otherNetworkIcon]} />
+					<View style={[styles.networkIcon, { backgroundColor: color || colors.red }]} />
 					<Text style={styles.networkName} testID={'navbar-title-network'}>
 						{name}
 					</Text>

diff --git a/app/components/UI/TransactionHeader/__snapshots__/index.test.js.snap b/app/components/UI/TransactionHeader/__snapshots__/index.test.js.snap
@@ -79,7 +79,7 @@ exports[`TransactionHeader should render correctly 1`] = `
             "width": 5,
           },
           Object {
-            "backgroundColor": "green",
+            "backgroundColor": "#ff4a8d",
           },
         ]
       }

diff --git a/app/components/UI/TransactionHeader/index.js b/app/components/UI/TransactionHeader/index.js
@@ -70,11 +70,7 @@ class TransactionHeader extends PureComponent {
 		/**
 		 * String representing the selected network
 		 */
-		networkType: PropTypes.string,
-		/**
-		 * Object representing the status of infura networks
-		 */
-		networkStatus: PropTypes.object
+		networkType: PropTypes.string
 	};
 
 	/**
@@ -83,8 +79,8 @@ class TransactionHeader extends PureComponent {
 	 * @return {element} - JSX view element
 	 */
 	renderNetworkStatusIndicator = () => {
-		const { networkType, networkStatus } = this.props;
-		const networkStatusIndicatorColor = networkStatus[networkType] === 'ok' ? 'green' : 'red';
+		const { networkType } = this.props;
+		const networkStatusIndicatorColor = (networkList[networkType] && networkList[networkType].color) || colors.red;
 		const networkStatusIndicator = (
 			<View style={[styles.networkStatusIndicator, { backgroundColor: networkStatusIndicatorColor }]} />
 		);