Bug 101726 - Sans config pulls in Microsoft Serifed font
Summary: Sans config pulls in Microsoft Serifed font
Status: RESOLVED FIXED
Alias: None
Product: fontconfig
Classification: Unclassified
Component: conf (show other bugs)
Version: unspecified
Hardware: All All
: medium major
Assignee: fontconfig-bugs
QA Contact: Behdad Esfahbod
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-07-08 21:49 UTC by Joseph Wang
Modified: 2017-07-11 04:20 UTC (History)
1 user (show)

See Also:
i915 platform:
i915 features:


Attachments
corrected non-latin file with correct sans configuration (8.56 KB, text/plain)
2017-07-08 21:49 UTC, Joseph Wang
Details
Patch replacing serifed microsoft fonts with sans fonts (822 bytes, patch)
2017-07-09 21:16 UTC, Joseph Wang
Details | Splinter Review
corrected non-latin file with correct sans configuration (816 bytes, patch)
2017-07-10 16:41 UTC, Joseph Wang
Details | Splinter Review

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.