Merge pull request #112 from MetaMask/AddLeadingZero

ui - formatBalance bug

CHANGELOG.md

@@ -5,6 +5,8 @@
 - Corrected text above account list. Selected account is visible to all sites, not just the current domain.
 - Merged the UI codebase into the main plugin codebase for simpler maintenance.
 - Fix Ether display rounding error. Now rendering to four decimal points.
+- Fix some inpage synchronous methods
+- Change account rendering to show four decimals and a leading zero.
 
 ## 1.5.0 2016-04-13
 

ui/app/util.js

@@ -71,7 +71,7 @@ function formatBalance(balance) {
   var padded = wei.toString(10)
   var len = padded.length
   var nonZeroIndex = padded.match(/[^0]/) && padded.match(/[^0]/).index
-  var beforeDecimal = padded.substr(nonZeroIndex ? nonZeroIndex : 0, len - 18)
+  var beforeDecimal = padded.substr(nonZeroIndex ? nonZeroIndex : 0, len - 18) || '0'
   var afterDecimal = padded.substr(len - 18, decimalsToKeep)
   return `${beforeDecimal}.${afterDecimal} ETH`
 }

ui/test/unit/util_test.js

@@ -71,7 +71,7 @@ describe('util', function() {
     it('should return eth as string followed by ETH', function() {
       var input = new ethUtil.BN(ethInWei, 10).div(new ethUtil.BN('2', 10)).toJSON()
       var result = util.formatBalance(input)
-      assert.equal(result, '.5000 ETH')
+      assert.equal(result, '0.5000 ETH')
     })
 
     it('should display four decimal points', function() {