From fd64c9004f5171d65d977d24168cd608ea1fae27 Mon Sep 17 00:00:00 2001 From: chrisgarrity <chrisg@media.mit.edu> Date: Mon, 23 Jul 2018 16:48:13 -0400 Subject: [PATCH] Temporarily exclude RTL languages from the menu MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Currently that’s just hebrew. It’s still available with the `?locale=he` for people to check translations. It’s being temporarily excluded from the menu until we have RTL support for the interface. --- .../language-selector/language-selector.jsx | 23 ++++++++++++------- 1 file changed, 15 insertions(+), 8 deletions(-) diff --git a/src/components/language-selector/language-selector.jsx b/src/components/language-selector/language-selector.jsx index ad7a9aa90..a6707bbce 100644 --- a/src/components/language-selector/language-selector.jsx +++ b/src/components/language-selector/language-selector.jsx @@ -5,6 +5,9 @@ import Box from '../box/box.jsx'; import locales from 'scratch-l10n'; import styles from './language-selector.css'; +// supported languages to exclude from the menu, but allow as a URL option +const ignore = ['he']; + class LanguageSelector extends React.Component { render () { const { @@ -26,14 +29,18 @@ class LanguageSelector extends React.Component { value={currentLocale} onChange={onChange} > - {Object.keys(locales).map(locale => ( - <option - key={locale} - value={locale} - > - {locales[locale].name} - </option> - ))} + { + Object.keys(locales) + .filter(l => !ignore.includes(l)) + .map(locale => ( + <option + key={locale} + value={locale} + > + {locales[locale].name} + </option> + )) + } </select> </div> </Box> -- GitLab