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 "menu" context for translation of menu labels #10932

Merged
merged 1 commit into from
Aug 31, 2021

Conversation

krassowski
Copy link
Member

@krassowski krassowski commented Aug 26, 2021

References

This is a follow up after #10931 and jupyterlab/jupyterlab-translate#8 adding "menu" context to the menu labels. This is needed for the translations to work (we lost the strings in pot file in transition the schema-based menus, and when re-introducing the them we decided to improve the translation context; this allows for example to translate "Edit" as "Edycja" (noun) for menu but "Edytuj" (verb) in another context, as appropriate.

Code changes

Use _p alias.

User-facing changes

It is the penultimate change needed to make the language packs cover all JupyterLab GUI 🤞

Backwards-incompatible changes

None (current language packs are missing labels for menus).

@krassowski krassowski added the bug label Aug 26, 2021
@krassowski krassowski added this to the 3.1.x milestone Aug 26, 2021
@jupyterlab-dev-mode
Copy link

Thanks for making a pull request to JupyterLab!

To try out this branch on binder, follow this link: Binder

Copy link
Member

@fcollonval fcollonval left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @krassowski

@krassowski krassowski requested a review from goanpeca August 29, 2021 10:47
Copy link
Member

@goanpeca goanpeca left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome :)

@fcollonval fcollonval merged commit 32ccdf1 into jupyterlab:master Aug 31, 2021
@fcollonval
Copy link
Member

@meeseeksdev please backport to 3.1.x

meeseeksmachine pushed a commit to meeseeksmachine/jupyterlab that referenced this pull request Aug 31, 2021
blink1073 pushed a commit that referenced this pull request Aug 31, 2021
…#10977)

Co-authored-by: Michał Krassowski <5832902+krassowski@users.noreply.github.com>
@github-actions github-actions bot added the status:resolved-locked Closed issues are locked after 30 days inactivity. Please open a new issue for related discussion. label Feb 28, 2022
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 28, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug pkg:mainmenu status:resolved-locked Closed issues are locked after 30 days inactivity. Please open a new issue for related discussion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants