Unified Diff: ext/background.js
Issue 4864767881641984:
Issue 1528 - Implemented backend for general tab of new options page (Closed)
Patch Set:
Created June 8, 2015, 4:09 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« background.js ('K')
|
« background.js ('k')
|
messageResponder.js » ('j')
|
no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: ext/background.js |
| =================================================================== |
| --- a/ext/background.js |
| +++ b/ext/background.js |
| @@ -76,8 +76,12 @@ |
| global.ext.PageMap = PageMap; |
| - global.ext.showOptions = function() |
| + global.ext.showOptions = function(callback) |
| { |
| - window.open("http://example.com/options.html", "_blank"); |
| + if (top.location.href.indexOf("options.html") == -1) |
| + window.open("options.html", "_blank"); |
| + |
| + if (callback) |
| + callback(); |
| }; |
| })(this); |
« background.js ('K')
|
« background.js ('k')
|
messageResponder.js » ('j')
|
no next file with comments »
