Issue 1678 - Added bidi support for Safari

safari/ext/common.js

Index: safari/ext/common.js
===================================================================
--- a/safari/ext/common.js
+++ b/safari/ext/common.js
@@ -104,8 +104,19 @@
     return candidates;
   };
 
+  var initCatalog = function(uiLocale)
+  {
+    var bidiDir = /^(ar|fa|he|ug|ur)(_|$)/.test(uiLocale) ? "rtl" : "ltr";

[Wladimir Palant, 2014/12/11 16:52:15]

I only realized now that we are looking at UI locale here rather than our
locale. What will happen if the UI locale is "ur" and we don't have that
translation? I guess that the user will see the fallback to English yet rendered
as RTL. Given that his browser UI is RTL, this might even make sense...

But I guess that this is a purely theoretical discussion. Sebastian mentioned
that Safari only supports ar and he, we have both of them covered.

[Sebastian Noack, 2014/12/11 17:03:05]

To be honest I didn't consider that case and agree that this code is
theoretically wrong, in terms that it considers the language of the browser,
instead the language of the translation used.

However, as you say, currently the only right-to-left languages supported by OS
X are Arabic and Hebrew. So this case won't happen in practice. Also note that
considering the language of the translation would require some additional
complexity since translations are loaded lazily on first use.

+    var catalog = Object.create(null);
+
+    catalog["@@ui_locale"] = [uiLocale, []];
+    catalog["@@bidi_dir" ] = [bidiDir,  []];
+
+    return catalog;
+  };
+
   var locales = getLocaleCandidates();
-  var catalog = {__proto__: null, "@@ui_locale": [locales[0], []]};
+  var catalog = initCatalog(locales[0]);
 
   var replacePlaceholder = function(text, placeholder, content)
   {