Unified Diff: ext/background.js
Issue 29488575:
Issue 5384 - Introduced dedicated mobile options page (Closed)
Patch Set: Removed menu item and fixed new linter error
Created Aug. 28, 2017, 1:12 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« background.js ('k')
|
locale/en-US/mobile-options.json » ('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 |
| @@ -80,7 +80,7 @@ |
| window.ext.showOptions = function(callback) |
| { |
| - if (top.location.href.indexOf("new-options.html") == -1) |
| + if (!/\/(?:mobile|new)-options\.html/.test(top.location.href)) |
| window.open("new-options.html", "_blank"); |
| if (callback) |
« no previous file with comments
|
« background.js ('k')
|
locale/en-US/mobile-options.json » ('j')
|
no next file with comments »
