Merge pull request #6943 from brave/cdc-fixes-2-17

Removing back arrow in asset detail view, updating top buy link [crypto.com widget 1.17]
brave · Oct 25, 2020 · bf39a75 · bf39a75
2 parents 9b26366 + 8531d77
commit bf39a75
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 3 deletions.
diff --git a/components/brave_new_tab_ui/components/default/cryptoDotCom/data.ts b/components/brave_new_tab_ui/components/default/cryptoDotCom/data.ts
@@ -38,7 +38,8 @@ const currencyNames = {
 
 const links = {
   'buyTop': 'https://auth.crypto.com/exchange/signup?utm_source=Brave&utm_medium=Trading%20Widget&utm_campaign=Brave%3AWW-en%3ATrading-Widget_MVP_InitialWidgetState_BuyButton_Top&utm_content=MVP_InitialWidgetState_BuyButton_Top',
-  'buyBottom': 'https://auth.crypto.com/exchange/signup?utm_source=Brave&utm_medium=Trading%20Widget&utm_campaign=Brave%3AWW-en%3ATrading-Widget_MVP_InitialWidgetState_BuyButton_Bottom&utm_content=MVP_InitialWidgetState_BuyButton_Bottom'
+  'buyBottom': 'https://auth.crypto.com/exchange/signup?utm_source=Brave&utm_medium=Trading%20Widget&utm_campaign=Brave%3AWW-en%3ATrading-Widget_MVP_InitialWidgetState_BuyButton_Bottom&utm_content=MVP_InitialWidgetState_BuyButton_Bottom',
+  'buyTopDetail': 'https://auth.crypto.com/exchange/signup?utm_source=Brave&utm_medium=Trading%20Widget&utm_campaign=Brave%3AWW-en%3ATrading-Widget_MVP_CryptoDetails_BuyButton_Top&utm_content=MVP_CryptoDetails_BuyButton_Top'
 }
 
 const dynamicBuyLink = (pair: string) => {

diff --git a/components/brave_new_tab_ui/components/default/cryptoDotCom/index.tsx b/components/brave_new_tab_ui/components/default/cryptoDotCom/index.tsx
@@ -175,6 +175,11 @@ class CryptoDotCom extends React.PureComponent<Props, State> {
     await this.props.onSetAssetData([asset])
   }
 
+  onClickBuyTopDetail = () => {
+    window.open(links.buyTopDetail, '_blank', 'noopener')
+    this.props.onBuyCrypto()
+  }
+
   onClickBuyTop = () => {
     window.open(links.buyTop, '_blank', 'noopener')
     this.props.onBuyCrypto()
@@ -341,7 +346,7 @@ class CryptoDotCom extends React.PureComponent<Props, State> {
             </Text>
           </FlexItem>
           <FlexItem $pl={5}>
-            <ActionButton onClick={this.onClickBuyPair.bind(this, `${currency}_USDT`)} small={true} light={true}>
+            <ActionButton onClick={this.onClickBuyTopDetail} small={true} light={true}>
               <UpperCaseText>
                 {getLocale('cryptoDotComWidgetBuy')}
               </UpperCaseText>
@@ -411,7 +416,10 @@ class CryptoDotCom extends React.PureComponent<Props, State> {
   }
 
   renderTitle () {
+    const { selectedAsset } = this.state
     const { optInMarkets, showContent } = this.props
+    const shouldShowBackArrow = !selectedAsset && showContent && optInMarkets
+
     return (
       <Header showContent={showContent}>
         <StyledTitle>
@@ -421,7 +429,7 @@ class CryptoDotCom extends React.PureComponent<Props, State> {
           <StyledTitleText>
             {'Crypto.com'}
           </StyledTitleText>
-          {showContent && optInMarkets &&
+          {shouldShowBackArrow &&
             <BackArrow marketView={true}>
               <CaratLeftIcon
                 onClick={this.optInMarkets.bind(this, false)}