Closed Bug 945074 Opened 11 years ago Closed 10 years ago

give nsTextFrames within continuing nsFirstLetter frames a style context that inherits from what the nsFirstLetter inherits from

Categories

(Core :: Layout, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla29

People

(Reporter: heycam, Assigned: heycam)

References

Details

Attachments

(1 file)

When we create a continuing nsTextFrame as part of the initial reflow of an nsFirstLetterFrame, we restyle the new nsTextFrame so that it doesn't have the ::first-line style.  This is done in nsFirstLetterFrame::DrainOverflowFrames.  However, we give it a style context that inherits from the continuing nsFirstLetterFrame, rather than inheriting from whatever the nsFirstLetterFrame inherits from.  Later on, a restyle will fix up this style context inheritance, but it would be better to get it right from the start.  (Especially as once bug 931668 is complete, we may not get to restyle the nsTextFrame.)
Attached patch patchSplinter Review
Attachment #8340845 - Flags: review?(dbaron)
Blocks: 931668
Comment on attachment 8340845 [details] [diff] [review]
patch

r=dbaron; sorry for the delay
Attachment #8340845 - Flags: review?(dbaron) → review+
OS: Mac OS X → All
Hardware: x86 → All
https://hg.mozilla.org/mozilla-central/rev/114c763763b0
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla29
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: