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

RESOLVED FIXED

Status

()

Core
Graphics
--
minor
RESOLVED FIXED
11 years ago
8 years ago

People

(Reporter: Jesse Ruderman, Unassigned)

Tracking

({regression, testcase})

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

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments, 1 obsolete attachment)

(Reporter)

Description

11 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

11 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+

Updated

8 years ago
Blocks: 578118
You need to log in before you can comment on or make changes to this bug.