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

Doc: Disable smartquotes for zh-tw and fr translations. #9337

Closed
wants to merge 2 commits into from

Conversation

JulienPalard
Copy link
Member

@JulienPalard
Copy link
Member Author

@adrianliaw can you please test a build with this patch and validate it fixes zh-tw quote issues?

@JulienPalard JulienPalard changed the title Doc: Disable smartquotes for zh-tw translations. Doc: Disable smartquotes for zh-tw and fr translations. Sep 15, 2018
@adrianliaw
Copy link
Contributor

It works perfectly for our build. Thank you @JulienPalard for the PR!

As a side note, previously the issue occurred in the translation of tutorial/venv.rst where:

pip has a number of subcommands: “search”, “install”, “uninstall”, “freeze”, etc.

was translated into:

pip 有好幾個子指令:"search"、"install"、"uninstall"、"freeze" 等等。

but output as

pip 有好幾個子指令:」search」、」install」、」uninstall」、」freeze」 等等。

Now by applying this PR, this issue doesn't raise again.

Copy link
Contributor

@willingc willingc left a comment

Choose a reason for hiding this comment

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

Looks good. Thanks @JulienPalard. I'll leave this for you to merge.

@JulienPalard
Copy link
Member Author

JulienPalard commented Oct 10, 2018

I prefer the #9423 version, so I close mine.

@JulienPalard JulienPalard deleted the smartquotes-zh-tw branch October 10, 2018 12:14
adrianliaw added a commit to adrianliaw/cpython that referenced this pull request Oct 11, 2018
This is a replacement for python#9337.

* Using conf.py to disable smartquotes through Sphinx instead of using docutils.conf
* Disable smartquotes for Japanese, French and Traditional Chinese translations

See:
* Suggesting to use conf.py: https://mail.python.org/pipermail/doc-sig/2018-September/004084.html
* Original smartquotes issue in ja translations: https://bugs.python.org/issue31793
* Disabling for ja: python#4006
* Smartquotes issue in fr: python/python-docs-fr#303
* Smartquotes issue in zh_TW: https://mail.python.org/pipermail/doc-sig/2018-August/004079.html
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.

5 participants