Fixes #226, combines disabled text strings

brave · Oct 23, 2018 · ac98f35 · ac98f35
1 parent 046ebcf
commit ac98f35
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 15 deletions.
diff --git a/src/features/rewards/mainToggle/__snapshots__/spec.tsx.snap b/src/features/rewards/mainToggle/__snapshots__/spec.tsx.snap
@@ -263,21 +263,16 @@ exports[`MainToggle tests basic tests matches the snapshot 1`] = `
       className="c14"
     >
       MISSING: rewardsOffText1
-       
-      <br />
-      <b>
-        MISSING: rewardsOffText2
-      </b>
     </div>
     <div
       className="c13"
     >
-      MISSING: rewardsOffText3
+      MISSING: rewardsOffText2
     </div>
     <div
       className="c14"
     >
-      MISSING: rewardsOffText4
+      MISSING: rewardsOffText3
     </div>
   </div>
 </div>

diff --git a/src/features/rewards/mainToggle/index.tsx b/src/features/rewards/mainToggle/index.tsx
@@ -47,12 +47,11 @@ export default class MainToggle extends React.PureComponent<Props, {}> {
           ? <StyledContent>
             <StyleTitle>{getLocale('rewardsWhy')}</StyleTitle>
             <StyleText>
-              {getLocale('rewardsOffText1')} <br/>
-              <b>{getLocale('rewardsOffText2')}</b>
+              {getLocale('rewardsOffText1')}
             </StyleText>
-            <StyleTitle>{getLocale('rewardsOffText3')}</StyleTitle>
+            <StyleTitle>{getLocale('rewardsOffText2')}</StyleTitle>
             <StyleText>
-              {getLocale('rewardsOffText4')}
+              {getLocale('rewardsOffText3')}
             </StyleText>
           </StyledContent>
           : null

diff --git a/stories/assets/locale.ts b/stories/assets/locale.ts
@@ -110,10 +110,9 @@ const locale: Record<string, string> = {
   rewardsContributeAttention: 'Attention',
   rewardsContributeAttentionScore: 'Your Attention Score',
   rewardsContributeText1: 'You’re currently supporting',
-  rewardsOffText1: 'Do you know that you’ve been paying for the web content with your data for the digital ads? You didn’t have a voice in it and worse, you’re exposed to privacy and security risks.',
-  rewardsOffText2: 'Brave Rewards allows you to take control back.',
-  rewardsOffText3: 'How does it work?',
-  rewardsOffText4: 'Your attention is valuable. Get paid for the ads. And pay directly the favorite content creators at your will. That way, they can grow and continue delivering the content that delights you.',
+  rewardsOffText1: 'Do you know that you’ve been paying for the web content with your data for the digital ads? You didn’t have a voice in it and worse, you’re exposed to privacy and security risks. Brave Rewards allows you to take control back.',
+  rewardsOffText2: 'How does it work?',
+  rewardsOffText3: 'Your attention is valuable. Get paid for the ads. And pay directly the favorite content creators at your will. That way, they can grow and continue delivering the content that delights you.',
   rewardsPanelEmptyText1: 'Sadly, no tokens yet.',
   rewardsPanelEmptyText2: '3 ways to fill your wallet:',
   rewardsPanelEmptyText3: 'You can add funds.',