Dynamic cross-element ligature leaves space between parts of the ligature

RESOLVED FIXED

Status

()

--
minor
RESOLVED FIXED
12 years ago
9 years ago

People

(Reporter: jruderman, Unassigned)

Tracking

({regression, testcase})

Trunk
x86
macOS
regression, testcase
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments, 1 obsolete attachment)

(Reporter)

Description

12 years ago
Created attachment 269494 [details]
testcase

Mozilla/5.0 (Macintosh; U; Intel Mac OS X; en-US; rv:1.9a6pre) Gecko/20070622 Minefield/3.0a6pre

In the testcase, you get the "fi" ligature form, but there are a few blank pixels between the "f" part and the "i" part.
(Reporter)

Comment 1

12 years ago
Created attachment 269495 [details]
better testcase

Easier to make a "sane" reftest pair if there's a <span> in there.  By "sane" I mean that Firefox 2 (which doesn't attempt to do cross-element ligatures at all) can pass.
Attachment #269494 - Attachment is obsolete: true
Created attachment 270119 [details]
slightly more difficult testcase

This testcase uses a nested span to hide the new element, which is slightly more difficult to fix (bug still fixed by the patch in bug 385607). I'll make these tests into reftests when that patch lands.
(Reporter)

Comment 3

11 years ago
Seems to have been fixed by bug 385607.  roc, do you still want to make the reftests, or should I?
Status: NEW → RESOLVED
Last Resolved: 11 years ago
Flags: in-testsuite?
Resolution: --- → FIXED
If you could, that would be great, thanks.
(Reporter)

Comment 5

11 years ago
Reftests checked in.
Flags: in-testsuite? → in-testsuite+
You need to log in before you can comment on or make changes to this bug.