Bug 101726

Summary: Sans config pulls in Microsoft Serifed font
Product: fontconfig Reporter: Joseph Wang <joequant>
Component: confAssignee: fontconfig-bugs
Status: RESOLVED FIXED QA Contact: Behdad Esfahbod <freedesktop>
Severity: major    
Priority: medium CC: akira
Version: unspecified   
Hardware: All   
OS: All   
Whiteboard:
i915 platform: i915 features:
Attachments: corrected non-latin file with correct sans configuration
Patch replacing serifed microsoft fonts with sans fonts
corrected non-latin file with correct sans configuration

Description Joseph Wang 2017-07-08 21:49:59 UTC
Created attachment 132566 [details]
corrected non-latin file with correct sans configuration

The config file 65-nonlatin.conf config for Sans pulls in MingLiu and SimSum Chinese fonts which are serifed.  This is unnecessary as Microsoft has produced some San Serif fonts, and in any case the config should fall back on the open source sans fonts for Chinese.

To reproduce switch to a Sans environment with Chinese, and you will see that the characters all have serifs.
Comment 1 Joseph Wang 2017-07-08 21:51:12 UTC
The corrected file uses microsoft JhengHei and YaHei fonts.

Cross reference to 

https://bugs.mageia.org/show_bug.cgi?id=21206
Comment 2 Akira TAGOH 2017-07-09 08:56:41 UTC
Please attach a file generated by git format-patch instead of the raw file.
Comment 3 Joseph Wang 2017-07-09 21:16:30 UTC
Created attachment 132577 [details] [review]
Patch replacing serifed microsoft fonts with sans fonts
Comment 4 Joseph Wang 2017-07-09 21:16:55 UTC
Patch up loaded
Comment 5 Akira TAGOH 2017-07-10 11:37:28 UTC
That change also affects and could be used for the purpose of non-UI rendering though, is using the UI variant really better choice?
Comment 6 Joseph Wang 2017-07-10 16:15:45 UTC
I'll try to see what happens if I take out the UI
Comment 7 Joseph Wang 2017-07-10 16:41:37 UTC
Created attachment 132590 [details] [review]
corrected non-latin file with correct sans configuration
Comment 8 Joseph Wang 2017-07-10 16:41:58 UTC
fixed patch with Non-UI
Comment 9 Akira TAGOH 2017-07-11 04:20:32 UTC
Thanks. merged into git.

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.