Bug 94038 - Poppler renders verdana bold font embedded as CID Truetype (subset) quite poorly
Summary: Poppler renders verdana bold font embedded as CID Truetype (subset) quite poorly
Status: RESOLVED FIXED
Alias: None
Product: poppler
Classification: Unclassified
Component: splash backend (show other bugs)
Version: unspecified
Hardware: x86-64 (AMD64) Linux (All)
: medium normal
Assignee: poppler-bugs
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-02-07 22:43 UTC by sergio.callegari
Modified: 2016-03-08 22:09 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments
assure line width > 0 in case of text stroke (760 bytes, patch)
2016-02-08 15:13 UTC, Thomas Freitag
Details | Splinter Review

Description sergio.callegari 2016-02-07 22:43:06 UTC
Opening a document containing the font in subject, its rendering with okular is extremely poor and almost unreadable. Acroread for linux renders the font just fine. KDE okular devels have diagnosed the problem to be in poppler.
See https://bugs.kde.org/show_bug.cgi?id=359118 for details and a sample document.

System is kubuntu 15.10 with poppler 0.33
Comment 1 Albert Astals Cid 2016-02-07 22:44:41 UTC
The document https://bugs.kde.org/attachment.cgi?id=97075
Comment 2 Albert Astals Cid 2016-02-07 22:44:59 UTC
Seems to be splash backend only.
Comment 3 Thomas Freitag 2016-02-08 15:13:28 UTC
Created attachment 121591 [details] [review]
assure line width > 0 in case of text stroke

The problem is here that the text render mode is set to 2, which means fill the text and then stroke with line width, but line width wasn't set, so it is 0.0. Then splash strokes with 1 pixel width without anti aliasing effects, which is quite poor.

The spec says for line width = 0: 

A line width of 0 shall denote the thinnest line that can be rendered at device resolution: 1 device pixel wide. However, some devices cannot reproduce 1-pixel lines, and on high-resolution devices, they are nearly invisible. Since the results of rendering such zero-width lines are device-dependent, they should not be used.

This patch now set the line width to 1 device pixel wide, but just in case of text drawing.
Comment 4 Thomas Freitag 2016-03-08 09:05:37 UTC
@Albert?
Comment 5 Albert Astals Cid 2016-03-08 22:09:19 UTC
Awesome as always. Commited.


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.