Issue 1668 - Fixed direction on add-on pages for right-to-left scripts

chrome/content/ui/i18n.js

 /*
  * This file is part of Adblock Plus <http://adblockplus.org/>,
  * Copyright (C) 2006-2014 Eyeo GmbH
  *
  * Adblock Plus is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 3 as
  * published by the Free Software Foundation.
  *
  * Adblock Plus is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
  * GNU General Public License for more details.
  *
  * You should have received a copy of the GNU General Public License
  * along with Adblock Plus.  If not, see <http://www.gnu.org/licenses/>.
  */
 
 var i18n;
-var locale;
 
 if (typeof ext != "undefined")
 {
   i18n = ext.i18n;
-  locale = ext.i18n.getMessage("@@ui_locale").replace(/_/g, "-");
+
+  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
 
   // Randomize URI to work around bug 719376
   var pageName = location.pathname.replace(/.*\//, '').replace(/\..*?$/, '');
   var stringBundle = Services.strings.createBundle("chrome://adblockplus/locale/" + pageName +
     ".properties?" + Math.random());
 
@@ skipping 42 matching lines @@
         }
         catch(e)
         {
           Cu.reportError(e);
           return "Missing translation: " + key;
         }
       }
     };
   })();
 
-  locale = require("utils").Utils.appLocale;
+  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
 // parsed as JSON and used as parameters to substitute into placeholders in the i18n
 // message.
 i18n.setElementText = function(element, stringName, arguments)
 {
   function processString(str, element)
   {
     var match = /^(.*?)<(a|strong)>(.*?)<\/\2>(.*)$/.exec(str);
@@ skipping 42 matching lines @@
   var d = new Date(when);
   var timeString = d.toLocaleTimeString();
 
   var now = new Date();
   if (d.toDateString() == now.toDateString())
     return [timeString];
   else
     return [timeString, d.toLocaleDateString()];
 }
 
-function onDOMLoaded()
-{
-  document.documentElement.lang = locale;
-
-  if (locale == "ar" || locale == "he")
-    document.documentElement.dir = "rtl";
-
-  loadI18nStrings();
-}
-
-window.addEventListener("DOMContentLoaded", onDOMLoaded, true);
+// Fill in the strings as soon as possible
+window.addEventListener("DOMContentLoaded", loadI18nStrings, true);