FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Make copybutton messages translatable by ccwang002 · Pull Request #27 · python/python-docs-theme · GitHub

Repository navigation

Make copybutton messages translatable - #27

Closed
ccwang002 wants to merge 1 commit into
python:mainfrom
ccwang002:translate-copybutton-msg
Closed

ccwang002 wants to merge 1 commit into
python:mainfrom
ccwang002:translate-copybutton-msg

Conversation

Copy link
Copy Markdown
Contributor

This PR implements #23 which make the messages of copybutton (>>>) translatable.

When I tested my change using the master branch of Python (branch 3.7 doesn't seem to use python-docs-theme), I couldn't get it read my modified layout.html. However, if I manually introduced the change, the pot files would include the new strings and the translation worked.

I think it is better to have someone more familiar with the project structure of PEP 545 to test my implementation before it is merged.

Copy link
Copy Markdown
Member

There's a little issue, the command gathering translations strings:

sphinx-build -Q -b gettext -D gettext_compact=0 Doc pot/

won't pick it up, so translation teams won't see it as a sentence to translate. I don't have the time right now to check if there's a simple way to solve this, maybe Sphinx has a way to look for translations strings in configured theme?

Copy link
Copy Markdown
Collaborator

These resources might be helpful:

  1. Add localization language support readthedocs/sphinx_rtd_theme#403
  2. Added localization language support readthedocs/sphinx_rtd_theme#405

Copy link
Copy Markdown
Contributor

I think I looked into this in the past and it was possible to directly translate the strings in the javascript file

python-cla-bot Bot commented Apr 6, 2025

Copy link
Copy Markdown

The following commit authors need to sign the Contributor License Agreement:

hugovk commented Apr 6, 2025

Copy link
Copy Markdown
Member

@tomasr8 Can this be closed now #225 is merged?

tomasr8 commented Apr 6, 2025

Copy link
Copy Markdown
Member

I believe so, the button title is now properly i18n'd.

hugovk closed this Apr 6, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

7 participants


Back | FazBrowse Home | New Git URL