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
Dynamic label for toggle sidebar element in keyboard shortcuts #16218
Dynamic label for toggle sidebar element in keyboard shortcuts #16218
Conversation
Thanks for making a pull request to jupyterlab! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for starting this! Two suggestions:
- the call to
trans.__
needs a slight adjustment as arguments do not get translated - the existing label should be kept as a fallback when args are not given because a command label can be also queried without arguments.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @itsmevichu!
…erlab#16218) * Dynamic label for toggle sidebar element in ketboard shortcuts * Adapted review comments. * Adapting review comments.
…erlab#16218) * Dynamic label for toggle sidebar element in ketboard shortcuts * Adapted review comments. * Adapting review comments.
Fix for #16217