Skip to content

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