Unified Diff: messageResponder.js
Issue 6289768170127360:
Issue 2067 - Unify logic retrieving the UI locale (Closed)
Patch Set:
Created March 1, 2015, 12:55 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« i18n.js ('k')
|
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: messageResponder.js |
| =================================================================== |
| --- a/messageResponder.js |
| +++ b/messageResponder.js |
| @@ -110,10 +110,10 @@ |
| callback(Utils.getDocLink(message.link)); |
| else if (message.what == "localeInfo") |
| { |
| - callback({ |
| - locale: Utils.appLocale, |
| - isRTL: Utils.chromeRegistry.isLocaleRTL("adblockplus") |
| - }); |
| + var bidiDir = ext.i18n.getMessage("@@bidi_dir"); |
|
Wladimir Palant
2015/03/02 11:40:51
ext.i18n is purely content side in Firefox, so thi
Sebastian Noack
2015/03/02 11:56:03
Oh, didn't know that.
|
| + if (!bidiDir) |
| + bidiDir = Utils.chromeRegistry.isLocaleRTL("adblockplus") ? "rtl" : "ltr"; |
| + callback({locale: Utils.appLocale, bidiDir: bidiDir}); |
| } |
| else |
| callback(null); |
