Fix drawing of partial ligatures

RESOLVED FIXED

Status

()

Core
Layout
RESOLVED FIXED
11 years ago
11 years ago

People

(Reporter: roc, Assigned: roc)

Tracking

Trunk
Points:
---
Bug Flags:
in-testsuite ?

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

Created attachment 266327 [details]
testcase (requires Zapfino and a new-textframe build)

There are two bugs with the drawing of partial ligatures, shown in the attached testcase (you'll need the full Zapfino font and the new textframe to see this). Select just the first character of the "Th" ligature to see...

1) The ascender of the h is cut off to the right of the ligature. We're not correctly detecting that the h is the end of the ligature. Simple off by one error.

2) We should pixel-snap the clip rect we use to draw partial ligatures. This will avoid seams between two components of the same ligature.
Flags: blocking1.9?

Updated

11 years ago
Attachment #266328 - Flags: review?(pavlov) → review+
checked in.
Status: NEW → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → FIXED

Updated

11 years ago
Flags: in-testsuite?

Updated

11 years ago
Flags: blocking1.9?
You need to log in before you can comment on or make changes to this bug.