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 language picker menu #411

Merged
merged 1 commit into from
Feb 15, 2023
Merged

Add language picker menu #411

merged 1 commit into from
Feb 15, 2023

Conversation

mgeisler
Copy link
Collaborator

@mgeisler mgeisler commented Feb 14, 2023

The picker is a drop-down menu using the same design as the theme picker in the top-left.

There doesn’t seem to be an easy way to pass in a list of languages and descriptions, so for now we’ll have to expand the menu by hand as we add new languages. A comment has been added to publish.yml to remind us of this.

Part of google/mdbook-i18n-helpers#35.

The picker is a drop-down menu using the same design as the theme
picker in the top-left.

There doesn’t seem to be an easy way to pass in a list of languages
and descriptions, so for now we’ll have to expand the menu by hand as
we add new languages. A comment has been added to `publish.yml` to
remind us of this.
@mgeisler
Copy link
Collaborator Author

To test this, you could download one or more of the translations from the test run and then look at them in a browser 🙂

@rastringer
Copy link
Collaborator

Amazing, thank you

@mgeisler mgeisler merged commit 3b7123d into main Feb 15, 2023
@mgeisler mgeisler deleted the language-picker branch February 15, 2023 14:10
NoahDragon pushed a commit to wnghl/comprehensive-rust that referenced this pull request Jul 19, 2023
The picker is a drop-down menu using the same design as the theme
picker in the top-left.

There doesn’t seem to be an easy way to pass in a list of languages
and descriptions, so for now we’ll have to expand the menu by hand as
we add new languages. A comment has been added to `publish.yml` to
remind us of this.
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.

2 participants