Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: chrome/content/ui/i18n.js

Issue 5137457368530944: Issue 1668 - Fixed direction on add-on pages for right-to-left scripts (Closed)
Patch Set: Use browser APIs do determine direction Created Dec. 9, 2014, 5:36 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/content/ui/firstRun.js ('k') | chrome/skin/firstRun.css » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/content/ui/i18n.js
===================================================================
--- a/chrome/content/ui/i18n.js
+++ b/chrome/content/ui/i18n.js
@@ -18,12 +18,12 @@
var i18n;
if (typeof ext != "undefined")
+{
i18n = ext.i18n;
-else if (typeof chrome != "undefined")
- // TODO: This check only exist for backwards compatibility, while the Safari
- // port isn't merged into the adblockpluschrome repo. So this branch should
- // be removed when the Safari port was merged.
- i18n = chrome.i18n;
+
+ document.documentElement.lang = ext.i18n.getMessage("@@ui_locale").replace(/_/g, "-");
+ document.documentElement.dir = ext.i18n.getMessage("@@bidi_dir");
+}
else
{
// Using Firefox' approach on i18n instead
@@ -84,6 +84,10 @@
}
};
})();
+
+ var Utils = require("utils").Utils;
+ document.documentElement.lang = Utils.appLocale;
+ document.documentElement.dir = Utils.chromeRegistry.isLocaleRTL("adblockplus") ? "rtl" : "ltr";
}
// Inserts i18n strings into matching elements. Any inner HTML already in the element is
« no previous file with comments | « chrome/content/ui/firstRun.js ('k') | chrome/skin/firstRun.css » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld