Show friendly message if wallet address & paymentURL are not available

Also fix vertical alignment of funds select box. Auditors: @mrose17
brave · Aug 25, 2016 · 5e0b679 · 5e0b679
1 parent 23502f6
commit 5e0b679
Show file tree

Hide file tree

Showing 3 changed files with 23 additions and 12 deletions.
diff --git a/app/extensions/brave/locales/en-US/preferences.properties b/app/extensions/brave/locales/en-US/preferences.properties
@@ -42,6 +42,7 @@ bitcoinQR=Brave wallet QR code:
 bitcoinCopyAddress=Copy Bitcoin address to clipboard
 bitcoinVisitAccount=Transfer BTC
 bitcoinBalance=Please transfer:&nbsp;
+bitcoinWalletNotAvailable=Wallet information not available. :(
 usd=&#36;
 done=Done
 enable=Enable

diff --git a/js/about/preferences.js b/js/about/preferences.js
@@ -295,21 +295,27 @@ class BitcoinDashboard extends ImmutableComponent {
         </div>
         <div className='panel'>
           <div className='settingsListTitle' data-l10n-id='bitcoinAdd' />
-          <img src={this.ledgerData.get('paymentIMG')} title='Brave wallet QR code' />
           {
-            this.ledgerData.get('hasBitcoinHandler')
+            this.ledgerData.get('address') && this.ledgerData.get('paymentURL')
               ? <div>
-                <a href={this.ledgerData.get('paymentURL')} target='_blank'>
-                  <Button l10nId='bitcoinVisitAccount' className='primaryButton' />
-                </a>
-                <div data-l10n-id='bitcoinAddress' className='labelText' />
-                <span className='fa fa-clipboard settingsListCopy alt' title='Copy to clipboard' onClick={this.copyToClipboard.bind(this, this.ledgerData.get('address'))} />
-                <span className='smallText'>{this.ledgerData.get('address')}</span>
-              </div>
-              : <div>
-                <div data-l10n-id='bitcoinPaymentURL' className='labelText' />
-                <span id='bitcoinPaymentURL' title='Copy to clipboard' onClick={this.copyToClipboard.bind(this, this.ledgerData.get('paymentURL'))}>{this.ledgerData.get('paymentURL')}</span>
+                <img src={this.ledgerData.get('paymentIMG')} title='Brave wallet QR code' />
+                {
+                  this.ledgerData.get('hasBitcoinHandler')
+                    ? <div>
+                      <a href={this.ledgerData.get('paymentURL')} target='_blank'>
+                        <Button l10nId='bitcoinVisitAccount' className='primaryButton' />
+                      </a>
+                      <div data-l10n-id='bitcoinAddress' className='labelText' />
+                      <span className='fa fa-clipboard settingsListCopy alt' title='Copy to clipboard' onClick={this.copyToClipboard.bind(this, this.ledgerData.get('address'))} />
+                      <span className='smallText'>{this.ledgerData.get('address')}</span>
+                    </div>
+                    : <div>
+                      <div data-l10n-id='bitcoinPaymentURL' className='labelText' />
+                      <span id='bitcoinPaymentURL' title='Copy to clipboard' onClick={this.copyToClipboard.bind(this, this.ledgerData.get('paymentURL'))}>{this.ledgerData.get('paymentURL')}</span>
+                    </div>
+                }
               </div>
+            : <div data-l10n-id='bitcoinWalletNotAvailable' />
           }
         </div>
       </div>

diff --git a/less/about/preferences.less b/less/about/preferences.less
@@ -751,3 +751,7 @@ table.sortableTable {
     margin-left: 15px;
   }
 }
+
+#fundsSelectBox {
+  margin-top: 32px;
+}