diff options
author | Dan Finlay <dan@danfinlay.com> | 2016-08-17 08:32:54 +0800 |
---|---|---|
committer | Dan Finlay <dan@danfinlay.com> | 2016-08-17 08:32:54 +0800 |
commit | a167bbc5a0f29568ec8e53ecdd942724aa15604b (patch) | |
tree | 0b63a2875718f692896af1cad3eca9fd393f15f9 /ui/app/conf-tx.js | |
parent | 030bdec27a95390207b9147c95b810893756db6d (diff) | |
download | tangerine-wallet-browser-a167bbc5a0f29568ec8e53ecdd942724aa15604b.tar.gz tangerine-wallet-browser-a167bbc5a0f29568ec8e53ecdd942724aa15604b.tar.zst tangerine-wallet-browser-a167bbc5a0f29568ec8e53ecdd942724aa15604b.zip |
MVP Popup Notifications Working
I'm unsure which will be more performant: A notification using a trimmed down version of the UI, or using them both, letting the browser cache them both.
In any case, here I've modified the normal UI to recognize when it's a popup, and change the UX accordingly in a few ways:
- Hide the menu bar
- Hide the back button from the notifications view.
- When confirming the last tx, close the window.
Diffstat (limited to 'ui/app/conf-tx.js')
-rw-r--r-- | ui/app/conf-tx.js | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/ui/app/conf-tx.js b/ui/app/conf-tx.js index db876dd9b..8c6a136bf 100644 --- a/ui/app/conf-tx.js +++ b/ui/app/conf-tx.js @@ -36,6 +36,7 @@ ConfirmTxScreen.prototype.render = function () { var unconfTxList = txHelper(unconfTxs, unconfMsgs) var index = state.index !== undefined ? state.index : 0 var txData = unconfTxList[index] || {} + var isNotification = window.METAMASK_UI_TYPE === 'notification' return ( @@ -43,9 +44,9 @@ ConfirmTxScreen.prototype.render = function () { // subtitle and nav h('.section-title.flex-row.flex-center', [ - h('i.fa.fa-arrow-left.fa-lg.cursor-pointer', { + !isNotification ? h('i.fa.fa-arrow-left.fa-lg.cursor-pointer', { onClick: this.goHome.bind(this), - }), + }) : null, h('h2.page-subtitle', 'Confirm Transaction'), ]), |