Unified Diff: lib/appIntegration.js
Issue 8487012:
Slightly improved notification for fennec (Closed)
Patch Set:
Created Oct. 2, 2012, 11:01 a.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« no previous file
|
no next file »
|
no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: lib/appIntegration.js |
| =================================================================== |
| --- a/lib/appIntegration.js |
| +++ b/lib/appIntegration.js |
| @@ -271,9 +271,13 @@ |
| { |
| if ("BrowserApp" in window && "selectedTab" in window.BrowserApp) |
| { |
| - window.NativeWindow.doorhanger.show(message, id, buttons, window.BrowserApp.selectedTab.id, |
| + let tabId = window.BrowserApp.selectedTab.id; |
| + window.NativeWindow.doorhanger.show(message, id, buttons, tabId, |
| { |
| - persistence: persistence |
| + // No navigation is happening after doorhanger is shown |
| + // so persistence needs to be reduced by one |
| + persistence: persistence-1, |
| + timeout: 5000 |
|
Wladimir Palant
2012/10/02 13:03:45
As discussed on IRC - looks like doorhanger notifi
|
| } |
| ); |
| } |
« no previous file with comments
|
« no previous file
|
no next file »
|
no next file with comments »
