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 /app/scripts/lib | |
| 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 'app/scripts/lib')
| -rw-r--r-- | app/scripts/lib/extension-instance.js | 2 | ||||
| -rw-r--r-- | app/scripts/lib/notifications.js | 3 |
2 files changed, 3 insertions, 2 deletions
diff --git a/app/scripts/lib/extension-instance.js b/app/scripts/lib/extension-instance.js index d284895bc..1098130e3 100644 --- a/app/scripts/lib/extension-instance.js +++ b/app/scripts/lib/extension-instance.js @@ -45,7 +45,7 @@ function Extension () { if (browser[api]) { _this[api] = browser[api] } - } + } catch (e) {} try { _this.api = browser.extension[api] diff --git a/app/scripts/lib/notifications.js b/app/scripts/lib/notifications.js index e6bdeff09..de9cf26e3 100644 --- a/app/scripts/lib/notifications.js +++ b/app/scripts/lib/notifications.js @@ -33,7 +33,8 @@ function createMsgNotification (state) { function showNotification() { extension.windows.create({ url:"notification.html", - type:"panel", + type:"detached_panel", + focused:true, width:360, height:500, }) |