Merge pull request #2433 from danjm/NewUI-flat-precision-improvements

[NewUI] Improves precision of a calculation in send, and in confirm
This commit is contained in:
Chi Kei Chan 2017-10-23 11:50:59 -07:00 committed by GitHub
commit 0cbaec16f3
2 changed files with 5 additions and 3 deletions

View File

@ -224,7 +224,7 @@ ConfirmSendToken.prototype.renderTotalPlusGas = function () {
]),
h('div.confirm-screen-section-column', [
h('div.confirm-screen-row-info', `${fiatAmount + fiatGas} ${currentCurrency}`),
h('div.confirm-screen-row-info', `${addCurrencies(fiatAmount, fiatGas)} ${currentCurrency}`),
h('div.confirm-screen-row-detail', `${addCurrencies(tokenAmount, tokenGas || '0')} ${symbol}`),
]),
])

View File

@ -2,7 +2,7 @@ const Component = require('react').Component
const h = require('react-hyperscript')
const inherits = require('util').inherits
const Identicon = require('../identicon')
const { conversionUtil } = require('../../conversion-util')
const { conversionUtil, multiplyCurrencies } = require('../../conversion-util')
module.exports = CurrencyDisplay
@ -40,7 +40,9 @@ CurrencyDisplay.prototype.getAmount = function (value) {
const { selectedToken } = this.props
const { decimals } = selectedToken || {}
const multiplier = Math.pow(10, Number(decimals || 0))
const sendAmount = '0x' + Number(value * multiplier).toString(16)
const sendAmount = multiplyCurrencies(value, multiplier, {toNumericBase: 'hex'})
return selectedToken
? sendAmount
: toHexWei(value)