From f4d58ebc705287788c9eaee6025269dd78b051a1 Mon Sep 17 00:00:00 2001 From: Dan Finlay Date: Wed, 4 May 2016 18:08:31 -0700 Subject: [PATCH] Move account export to subview Account detail view now has an animated transitioning `subview` section that allows us to show extra details within it. Clicking `export` now slide replaces the transaction list with the export UI. Added cancel/done/submit buttons to the Export UI. Done submits like Enter did, the other two transition back to the transaction list. For some reason when first unlocking, the selected account is being instantly replaced with the accounts list, so I need to fix that before merging this into master. --- ui/app/account-detail.js | 106 ++++++++++---------------- ui/app/components/account-export.js | 87 +++++++++++++++++++++ ui/app/components/transaction-list.js | 2 +- ui/app/reducers/app.js | 17 +++++ 4 files changed, 145 insertions(+), 67 deletions(-) create mode 100644 ui/app/components/account-export.js diff --git a/ui/app/account-detail.js b/ui/app/account-detail.js index a876b275f..208ecc63a 100644 --- a/ui/app/account-detail.js +++ b/ui/app/account-detail.js @@ -1,11 +1,15 @@ const inherits = require('util').inherits +const extend = require('xtend') const Component = require('react').Component const h = require('react-hyperscript') const connect = require('react-redux').connect const copyToClipboard = require('copy-to-clipboard') const actions = require('./actions') +const ReactCSSTransitionGroup = require('react-addons-css-transition-group') + const AccountPanel = require('./components/account-panel') const transactionList = require('./components/transaction-list') +const ExportAccountView = require('./components/account-export') module.exports = connect(mapStateToProps)(AccountDetailScreen) @@ -78,12 +82,13 @@ AccountDetailScreen.prototype.render = function() { }, 'EXPORT'), ]), - transactionList(transactions - .filter(tx => tx.txParams.from === state.address) - .filter(tx => tx.txParams.metamaskNetworkId === state.networkVersion) - .sort((a, b) => b.time - a.time), state.networkVersion), - this.exportedAccount(accountDetail), - + h(ReactCSSTransitionGroup, { + transitionName: "main", + transitionEnterTimeout: 300, + transitionLeaveTimeout: 300, + }, [ + this.subview(), + ]), // transaction table /* h('section.flex-column', [ @@ -94,72 +99,41 @@ AccountDetailScreen.prototype.render = function() { ) } +AccountDetailScreen.prototype.subview = function() { + var subview + try { + subview = this.props.accountDetail.subview + } catch (e) { + subview = null + } + + switch (subview) { + case 'transactions': + return this.transactionList() + case 'export': + var state = extend({key: 'export'}, this.props) + return h(ExportAccountView, state) + default: + return this.transactionList() + } +} + +AccountDetailScreen.prototype.transactionList = function() { + var state = this.props + var transactions = state.transactions + + return transactionList(transactions + .filter(tx => tx.txParams.from === state.address) + .filter(tx => tx.txParams.metamaskNetworkId === state.networkVersion) + .sort((a, b) => b.time - a.time), state.networkVersion) +} + AccountDetailScreen.prototype.navigateToAccounts = function(event){ event.stopPropagation() this.props.dispatch(actions.showAccountsPage()) } -AccountDetailScreen.prototype.exportAccount = function(address) { - this.props.dispatch(actions.exportAccount(address)) -} - AccountDetailScreen.prototype.requestAccountExport = function() { this.props.dispatch(actions.requestExportAccount()) } -AccountDetailScreen.prototype.exportedAccount = function(accountDetail) { - if (!accountDetail) return - var accountExport = accountDetail.accountExport - - var notExporting = accountExport === 'none' - var exportRequested = accountExport === 'requested' - var accountExported = accountExport === 'completed' - - if (notExporting) return - - if (exportRequested) { - var warning = `Exporting your private key is very dangerous, - and you should only do it if you know what you're doing.` - var confirmation = `If you're absolutely sure, type "I understand" below and - hit Enter.` - return h('div', {}, [ - h('p.error', warning), - h('p', confirmation), - h('input#exportAccount', { - onKeyPress: this.onExportKeyPress.bind(this), - }) - ]) - } - - if (accountExported) { - return h('div.privateKey', { - - }, [ - h('label', 'Your private key (click to copy):'), - h('p.error.cursor-pointer', { - style: { - textOverflow: 'ellipsis', - overflow: 'hidden', - webkitUserSelect: 'text', - width: '100%', - }, - onClick: function(event) { - copyToClipboard(accountDetail.privateKey) - } - }, accountDetail.privateKey), - ]) - } -} - -AccountDetailScreen.prototype.onExportKeyPress = function(event) { - if (event.key !== 'Enter') return - event.preventDefault() - - var input = document.getElementById('exportAccount') - if (input.value === 'I understand') { - this.props.dispatch(actions.exportAccount(this.props.address)) - } else { - input.value = '' - input.placeholder = 'Please retype "I understand" exactly.' - } -} diff --git a/ui/app/components/account-export.js b/ui/app/components/account-export.js new file mode 100644 index 000000000..f79a533ba --- /dev/null +++ b/ui/app/components/account-export.js @@ -0,0 +1,87 @@ +const Component = require('react').Component +const h = require('react-hyperscript') +const inherits = require('util').inherits +const actions = require('../actions') + +module.exports = ExportAccountView + + +inherits(ExportAccountView, Component) +function ExportAccountView() { + Component.call(this) +} + +ExportAccountView.prototype.render = function() { + console.log("EXPORT VIEW") + console.dir(this.props) + var state = this.props + var accountDetail = state.accountDetail + + if (!accountDetail) return h('div') + var accountExport = accountDetail.accountExport + + var notExporting = accountExport === 'none' + var exportRequested = accountExport === 'requested' + var accountExported = accountExport === 'completed' + + if (notExporting) return h('div') + + if (exportRequested) { + var warning = `Exporting your private key is very dangerous, + and you should only do it if you know what you're doing.` + var confirmation = `If you're absolutely sure, type "I understand" below and + submit.` + return h('div', { key: 'exporting' }, [ + h('p.error', warning), + h('p', confirmation), + h('input#exportAccount', { + onKeyPress: this.onExportKeyPress.bind(this), + }), + h('button', { + onClick: () => this.onExportKeyPress({ key: 'Enter', preventDefault: () => {} }), + }, 'Submit'), + h('button', { + onClick: () => this.props.dispatch(actions.backToAccountDetail(this.props.address)) + }, 'Cancel'), + ]) + } + + if (accountExported) { + return h('div.privateKey', { + + }, [ + h('label', 'Your private key (click to copy):'), + h('p.error.cursor-pointer', { + style: { + textOverflow: 'ellipsis', + overflow: 'hidden', + webkitUserSelect: 'text', + width: '100%', + }, + onClick: function(event) { + copyToClipboard(accountDetail.privateKey) + } + }, accountDetail.privateKey), + h('button', { + onClick: () => this.props.dispatch(actions.backToAccountDetail(this.props.address)) + }, 'Done'), + ]) + } +} + +ExportAccountView.prototype.onExportKeyPress = function(event) { + if (event.key !== 'Enter') return + event.preventDefault() + + var input = document.getElementById('exportAccount') + if (input.value === 'I understand') { + this.props.dispatch(actions.exportAccount(this.props.address)) + } else { + input.value = '' + input.placeholder = 'Please retype "I understand" exactly.' + } +} + +ExportAccountView.prototype.exportAccount = function(address) { + this.props.dispatch(actions.exportAccount(address)) +} diff --git a/ui/app/components/transaction-list.js b/ui/app/components/transaction-list.js index 99a325e35..90f2955e6 100644 --- a/ui/app/components/transaction-list.js +++ b/ui/app/components/transaction-list.js @@ -4,7 +4,7 @@ const addressSummary = require('../util').addressSummary const explorerLink = require('../../lib/explorer-link') module.exports = function(transactions, network) { - return h('details', [ + return h('details', { key: 'transaction-list' }, [ h('summary', [ h('div.font-small', {style: {display: 'inline'}}, 'Transactions'), diff --git a/ui/app/reducers/app.js b/ui/app/reducers/app.js index 14f92a8c5..35ddf6759 100644 --- a/ui/app/reducers/app.js +++ b/ui/app/reducers/app.js @@ -23,6 +23,9 @@ function reduceApp(state, action) { var appState = extend({ currentView: seedWords ? seedConfView : defaultView, + accountDetail: { + subview: 'transactions', + }, currentDomain: 'example.com', transForward: true, // Used to render transition direction isLoading: false, // Used to display loading indicator @@ -131,6 +134,7 @@ function reduceApp(state, action) { return extend(appState, { currentView: {}, accountDetail: { + subview: 'transactions', accountExport: 'none', privateKey: '', }, @@ -144,6 +148,7 @@ function reduceApp(state, action) { context: action.value || account, }, accountDetail: { + subview: 'transactions', accountExport: 'none', privateKey: '', }, @@ -157,6 +162,7 @@ function reduceApp(state, action) { context: action.value, }, accountDetail: { + subview: 'transactions', accountExport: 'none', privateKey: '', }, @@ -218,6 +224,9 @@ function reduceApp(state, action) { name: 'accountDetail', context: state.metamask.selectedAddress, }, + accountDetail: { + subview: 'transactions', + }, }) } @@ -285,7 +294,13 @@ function reduceApp(state, action) { case actions.REQUEST_ACCOUNT_EXPORT: return extend(appState, { + transForward: true, + currentView: { + name: 'accountDetail', + context: appState.currentView.context, + }, accountDetail: { + subview: 'export', accountExport: 'requested', }, }) @@ -293,6 +308,7 @@ function reduceApp(state, action) { case actions.EXPORT_ACCOUNT: return extend(appState, { accountDetail: { + subview: 'export', accountExport: 'completed', }, }) @@ -300,6 +316,7 @@ function reduceApp(state, action) { case actions.SHOW_PRIVATE_KEY: return extend(appState, { accountDetail: { + subview: 'export', accountExport: 'completed', privateKey: action.value, },