Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork32k
Add 'Show translation source' to docs sidebar#130355
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
base:main
Are you sure you want to change the base?
Uh oh!
There was an error while loading.Please reload this page.
Conversation
Uh oh!
There was an error while loading.Please reload this page.
Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>
Uh oh!
There was an error while loading.Please reload this page.
m-aciek commentedFeb 20, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
I suggest to change the PR title to make it clear it's about the docs. |
I'm -1 on this. A |
I often use "show source" when I want to check, well, the source, for example if there's something amiss. If there's a translation that's missing or fixing, is the .po file the file that needs editing? If so, this would make it easy to click through and make a quick PR. |
Hugo already mentioned the main point. Another thing is the current „show source” button is useless in the translations as it is the English source. This will contributions as it is easy to find where the files are as the repo is not linked anywhere else in the docs! And also, Adam, you don’t have to worry about seeing it :-) |
Uh oh!
There was an error while loading.Please reload this page.
m-aciek commentedMar 4, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
I personally use "Show source" very rarely. But I think "Show translation source" would be as useful as "Show source" is, for translated content. |
Is there anything else left to do here@hugovk ? |
Friendly ping, so that we can get this in. |
Adam is -1, let me check in with him. We're not in a rush to get this in before the beta feature freeze, it can also go in after and be backported. |
That question wasn’t answered directly (assuming it wasn’t just rhetorical). Yes, the .po file should be edited in case of a missing or incorrect translation. That’s a good point—using PR-based translation workflows, this should be a convenient change. |
We could usePlausible custom events to track the usage of the show source link. Though it looks like it would require changes in both theme and the show source customization in the CPython's repo. |
Uh oh!
There was an error while loading.Please reload this page.
Discussionsphinx-doc/sphinx#13301
📚 Documentation preview 📚:https://cpython-previews--130355.org.readthedocs.build/