Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add a language toggle to NoRent. #1503

Merged
merged 3 commits into from
May 29, 2020
Merged

Add a language toggle to NoRent. #1503

merged 3 commits into from
May 29, 2020

Conversation

toolness
Copy link
Collaborator

@toolness toolness commented May 29, 2020

This adds a language toggle to the footer of NoRent. I put it in the footer instead of the navbar because the navbar was becoming quite cluttered on desktop, and on mobile it'd be hidden behind a hamburger menu; it's also based on a similar placement on websites like mozilla.org.

image

It's not exactly highly visible but my hope is that Django's auto-redirection of browsers to the appropriate locale based on their language preferences will take care of sending most users to the appropriate locale. (That is, if a user's browser is in Spanish and they go to https://norent.org/, they will be transparently redirected to https://norent.org/es/.)

Furthermore, it seems like this kind of solution isn't necessarily mutually exclusive with putting some kind of UI at the top of the page: as with the navbar links, we can have them both at the header and footer.

@toolness toolness merged commit 942ab50 into master May 29, 2020
@toolness toolness deleted the language-toggle branch May 29, 2020 16:10
@toolness toolness mentioned this pull request May 29, 2020
33 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant