Merge remote-tracking branch 'origin/develop' into Version-v8.0.0

* origin/develop:
  Fix connect long URL styling (#8719)
  Add nonce to transaction details (#8716)
feature/default_network_editable
Mark Stacey 5 years ago
commit 07dee72efd
  1. 2
      test/e2e/metamask-ui.spec.js
  2. 8
      ui/app/components/app/permissions-connect-header/index.scss
  3. 2
      ui/app/components/app/permissions-connect-header/permissions-connect-header.component.js
  4. 14
      ui/app/components/app/transaction-breakdown/transaction-breakdown.component.js
  5. 1
      ui/app/components/app/transaction-list-item-details/transaction-list-item-details.component.js

@ -474,7 +474,7 @@ describe('MetaMask', function () {
const popoverCloseButton = await driver.findClickableElement(By.css('.popover-header__button'))
const txGasPrices = await driver.findElements(By.css('.transaction-breakdown__value'))
const txGasPriceLabels = await driver.findElements(By.css('.transaction-breakdown-row__title'))
await driver.wait(until.elementTextMatches(txGasPrices[3], /^10$/), 10000)
await driver.wait(until.elementTextMatches(txGasPrices[4], /^10$/), 10000)
assert(txGasPriceLabels[2])
await popoverCloseButton.click()
})

@ -38,6 +38,14 @@
color: $Grey-500;
}
&__text {
width: 100%;
text-overflow: ellipsis;
overflow: hidden;
/*rtl:ignore*/
direction: rtl;
}
&__subtitle {
margin-top: 4px;
}

@ -22,7 +22,7 @@ export default class PermissionsConnectHeader extends Component {
return (
<div className="permissions-connect-header__icon">
<IconWithFallBack icon={ icon } name={ iconName } />
<div className="permissions-connect-header__text">{iconName }</div>
<div className="permissions-connect-header__text">{iconName}</div>
</div>
)
}

@ -16,6 +16,7 @@ export default class TransactionBreakdown extends PureComponent {
className: PropTypes.string,
nativeCurrency: PropTypes.string,
showFiat: PropTypes.bool,
nonce: PropTypes.string,
gas: PropTypes.oneOfType([PropTypes.string, PropTypes.number]),
gasPrice: PropTypes.oneOfType([PropTypes.string, PropTypes.number]),
value: PropTypes.oneOfType([PropTypes.string, PropTypes.number]),
@ -29,13 +30,22 @@ export default class TransactionBreakdown extends PureComponent {
render () {
const { t } = this.context
const { gas, gasPrice, value, className, nativeCurrency, showFiat, totalInHex, gasUsed } = this.props
const { gas, gasPrice, value, className, nonce, nativeCurrency, showFiat, totalInHex, gasUsed } = this.props
return (
<div className={classnames('transaction-breakdown', className)}>
<div className="transaction-breakdown__title">
{ t('transaction') }
</div>
<TransactionBreakdownRow title="Nonce">
{typeof nonce !== 'undefined'
? (
<HexToDecimal
className="transaction-breakdown__value"
value={nonce}
/>
) : null
}
</TransactionBreakdownRow>
<TransactionBreakdownRow title={t('amount')}>
<UserPreferencedCurrencyDisplay
className="transaction-breakdown__value"

@ -246,6 +246,7 @@ export default class TransactionListItemDetails extends PureComponent {
</div>
<div className="transaction-list-item-details__cards-container">
<TransactionBreakdown
nonce={transactionGroup.initialTransaction.txParams.nonce}
transaction={transaction}
className="transaction-list-item-details__transaction-breakdown"
/>

Loading…
Cancel
Save