Skip to content

[MASTER] Temporarily exclude RTL languages from the menu#2706

Merged
chrisgarrity merged 1 commit into
scratchfoundation:masterfrom
chrisgarrity:feature/exclude-rtl
Jul 23, 2018
Merged

[MASTER] Temporarily exclude RTL languages from the menu#2706
chrisgarrity merged 1 commit into
scratchfoundation:masterfrom
chrisgarrity:feature/exclude-rtl

Conversation

@chrisgarrity
Copy link
Copy Markdown
Contributor

Right to left (RTL) languages are not yet fully supported in the editor, so we don't want to give the impression they are by including them in the menu.

Translators can still load the language with the ?locale=he URL argument.

Currently the excluded languages is just 'he', but I wanted to write it in such a way that it can be used for multiple languages later if we need it.

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.
@rschamp
Copy link
Copy Markdown
Contributor

rschamp commented Jul 23, 2018

@chrisgarrity Don't forget to also merge this into develop if you haven't already.

@chrisgarrity chrisgarrity changed the title [HOTFIX] Temporarily exclude RTL languages from the menu [MASTER] Temporarily exclude RTL languages from the menu Jul 23, 2018
@chrisgarrity chrisgarrity merged commit a810f50 into scratchfoundation:master Jul 23, 2018
@chrisgarrity chrisgarrity deleted the feature/exclude-rtl branch July 24, 2018 14:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants