2 files changed, 34 insertions, 31 deletions
diff --git a/public/js/extra.js b/public/js/extra.js
index 10c3735b..0ba09388 100644
--- a/public/js/extra.js
+++ b/public/js/extra.js
@@ -1,6 +1,6 @@
 /* eslint-env browser, jquery */
 /* eslint no-console: ["error", { allow: ["warn", "error"] }] */
-/* global moment, serverurl, Cookies */
+/* global moment, serverurl */
 
 import Prism from 'prismjs'
 import hljs from 'highlight.js'
@@ -27,15 +27,11 @@ require('prismjs/components/prism-makefile')
 require('prismjs/components/prism-gherkin')
 
 require('./lib/common/login')
+require('./locale')
 require('../vendor/md-toc')
 var Viz = require('viz.js')
 const ui = getUIElements()
 
-if (Cookies.get('locale')) {
-  const lang = Cookies.get('locale')
-  moment.locale(lang)
-}
-
 // auto update last change
 window.createtime = null
 window.lastchangetime = null
diff --git a/public/js/locale.js b/public/js/locale.js
index 05c7b3e3..aca35b98 100644
--- a/public/js/locale.js
+++ b/public/js/locale.js
@@ -1,33 +1,40 @@
 /* eslint-env browser, jquery */
 /* global Cookies */
 
-var lang = 'en'
-var userLang = navigator.language || navigator.userLanguage
-var userLangCode = userLang.split('-')[0]
-var locale = $('.ui-locale')
-var supportLangs = []
-$('.ui-locale option').each(function () {
-  supportLangs.push($(this).val())
-})
-if (Cookies.get('locale')) {
-  lang = Cookies.get('locale')
-  if (lang === 'zh') {
-    lang = 'zh-TW'
+const supported = ['en', 'zh-CN', 'zh-TW', 'fr', 'de', 'ja', 'es', 'ca', 'el', 'pt', 'it', 'tr', 'ru', 'nl', 'hr', 'pl', 'uk', 'hi', 'sv', 'eo', 'da', 'ko', 'id', 'sr', 'vi', 'ar', 'cs', 'sk']
+
+function detectLang () {
+  if (Cookies.get('locale')) {
+    let lang = Cookies.get('locale')
+    if (lang === 'zh') {
+      lang = 'zh-TW'
+    }
+    return lang
+  }
+  const userLang = navigator.language || navigator.userLanguage
+  const userLangCode = userLang.split('-')[0]
+  if (supported.includes(userLangCode)) {
+    return userLangCode
+  } else if (supported.includes(userLang)) {
+    return userLang
   }
-} else if (supportLangs.indexOf(userLang) !== -1) {
-  lang = supportLangs[supportLangs.indexOf(userLang)]
-} else if (supportLangs.indexOf(userLangCode) !== -1) {
-  lang = supportLangs[supportLangs.indexOf(userLangCode)]
+  return 'en'
 }
 
-locale.val(lang)
-window.moment.locale(lang)
-$('select.ui-locale option[value="' + lang + '"]').attr('selected', 'selected')
+const lang = detectLang()
+const localeSelector = $('.ui-locale')
 
-locale.change(function () {
-  Cookies.set('locale', $(this).val(), {
-    expires: 365,
-    sameSite: 'strict'
+// the following condition is needed as the selector is only available in the intro/history page
+if (localeSelector.length > 0) {
+  localeSelector.val(lang)
+  $('select.ui-locale option[value="' + lang + '"]').attr('selected', 'selected')
+  localeSelector.change(function () {
+    Cookies.set('locale', $(this).val(), {
+      expires: 365,
+      sameSite: 'strict'
+    })
+    window.location.reload()
   })
-  window.location.reload()
-})
+}
+
+window.moment.locale(lang)