Twenty Seventeen: Error font-family style for Simplified Chinese (zh_CN).

[ze3kr]

In wordpress 5.0 (5.0-RC1-43944):

/wp-content/themes/twentyseventeen/assets/css/editor-blocks.css （For the new block editor）

Line 160:

/* Typography for Chinese Font */

html[lang^="zh-"] .edit-post-visual-editor .editor-block-list__block,
html[lang="zh-CN"] .edit-post-visual-editor .editor-block-list__block {
	font-family: "PingFang TC", "Helvetica Neue", Helvetica, STHeitiTC-Light, Arial, sans-serif;
}

zh-CN should use SC, but not TC. Or the type of character is not correct. Similar to: #39379

Should be changed to:

/* Typography for Chinese Font */

html[lang^="zh-"] .edit-post-visual-editor .editor-block-list__block {
	font-family: "PingFang TC", "Helvetica Neue", Helvetica, STHeitiTC-Light, Arial, sans-serif;
}

html[lang="zh-CN"] .edit-post-visual-editor .editor-block-list__block {
	font-family: "PingFang SC", "Helvetica Neue", Helvetica, STHeitiSC-Light, Arial, sans-serif;
}

[ze3kr]

I have also checked 2019 theme, and it does NOT have the same problem. It has set the font family for zh-CN correctly.

[vaishalipanchal]

Hi @ze3kr, Welcome to WordPress Trac! Thank you for your ticket.

As per your ticket if above Chinese(zh_CN) font issue is in Twenty Seventeen theme editor-blocks.css then theme is also use STHeitiTC-Light font family so needs to fix that also.

To contribute with patch refer this article - ​https://make.wordpress.org/core/handbook/contribute/git/#patches

[vaishalipanchal]

Patch.

[ze3kr]

Hi, I think the patch is incorrect (for both 2 files). And the original wp-content/themes/twentyseventeen/style.css is fine.

[ze3kr]

For Chinese other than Simplified Chinese, it should use TC:

font-family: "PingFang TC", "Helvetica Neue", Helvetica, STHeitiTC-Light, Arial, sans-serif;

For Simplified Chinese, it should use SC.

font-family: "PingFang SC", "Helvetica Neue", Helvetica, STHeitiSC-Light, Arial, sans-serif;

Note that there are 2 fonts need to specify SC and TC, PingFang and STHeiti.

So the style.css is fine(fixed in #39379 ), but not the editor-blocks.css.

[vaishalipanchal]

Updated patch.

[ze3kr]

45408.2.patch misses the "PingFang SC" on line 163

"PingFang TC" -> "PingFang SC"

[vaishalipanchal]

Updated patch.

[laurelfulford]

Thank you, @ze3kr, for the report and review, and @vaishalipanchal for the patch!

I can confirm that this patch makes these styles in editor-blocks.css line up with what is used in style.css: [lang^="zh-"] now uses PingFang TC and STHeitiTC-Light, and [lang="zh-CN"] now uses PingFang SC and STHeitiSC-Light.

[laurelfulford]

In 44235:

Twenty Seventeen: Update font-family for Simplified Chinese (zh_CN).

Fix the font-family used for Simplified Chinese (zh_CN) in the block editor CSS.

Props vaishalipanchal, ze3kr.
Fixes #45408.

[laurelfulford]

Reopening and marking fixed-5.0 so that this can be properly merged into trunk.

[desrosj]

In 44307:

Twenty Seventeen: Bug Fixes.

• Remove unnecessary ol styles from editor. Twenty Seventeen was designed with bolded numbers for its ordered list styles. This was removed from the theme prior to launch, but it was left in the editor styles, so it needed to be removed there, too.
• To improve accessibility, the cookies content checkbox in the comment form needed some visible indication when it’s in focus.
• Twenty Seventeen’s editor styles contained an Internet Explorer 8 specific fix that stopped images from being scalable in the editor in other browsers. Since the editor no longer supports IE8, this fix can safely be removed.
• Fix the font-family used for Simplified Chinese (zh_CN) in the block editor CSS.

Props mmaumio, edpittol, pratikthink, afercia, audrasjb, mihaivalentin, subrataemfluence, vaishalipanchal, ze3kr, laurelfulford.

Merges [44203-44205] and [44235] into trunk.

Fixes #44775, #44699, #39738, #45408.