Duplicate code.. Make a function.

jdm · Nov 3, 2014 · ae9713b · ae9713b
1 parent 4d72ba1
commit ae9713b
Showing 1 changed file with 10 additions and 10 deletions.
diff --git a/foopy.js b/foopy.js
@@ -6,6 +6,14 @@
     var currentLang = "en"; // Default lang
     var rtlLangs    = {"ar": "","fa": ""};
 
+    function detectRtl(value) {
+      if (value in rtlLangs) {
+        document.documentElement.dir = "rtl";
+      } else {
+        document.documentElement.dir = "ltr";
+      }
+    }
+
     function chooseNegativeResponse() {
         var responses = $('.negative').not('.visible');
 
@@ -127,11 +135,7 @@
 
     function onLangChange() {
         document.webL10n.setLanguage(this.value);
-        if (this.value in rtlLangs) {
-          document.documentElement.dir = "rtl";
-        } else {
-          document.documentElement.dir = "ltr";
-        }
+        detectRtl(this.value);
         setLangQueryString(this.value)
     }
 
@@ -204,11 +208,7 @@
         // If the browser language is supported, select the good option
 
         document.webL10n.setLanguage(value);
-        if (value in rtlLangs) {
-          document.documentElement.dir = "rtl";
-        } else {
-          document.documentElement.dir = "ltr";
-        }
+        detectRtl(value);
         option.prop('selected', 'selected');
 
         currentLang = value;