Bug 12691 – Generated content and display: table-cell duplicates content on font size change

• Component: CSS
• Status: RESOLVED FIXED

[Elliott Sprehn]

An element that has display: table-cell and generated content (with :before or :after) will regenerate this content after a font size change.

Using Apple++ or Apple+- to increase and decrease the font size on the linked page the content specified in the css will be generated again with each increase or decrease in the font size.

This occurs when font is changed by Javascript or the User.

Reproducible with both Konqueror and Safari, official 2.0.4 (419.3) and latest nightly (Thu Feb 8 6:10:29 GMT 2007) builds.

[Elliott Sprehn]

Created attachment 13061 [details]
Test Case

[Dave Hyatt]

Created attachment 14282 [details]
Patch to fix the problem.

This patch fixes the problem by making sure we drill into anonymous containers to find generated content that may have been wrapped.

[Dave Hyatt]

Comment on attachment 14282 [details]
Patch to fix the problem.

There's a minor problem.  Need to fix.

[Dave Hyatt]

Created attachment 14283 [details]
Better version of patch.

Fixed a mistake made later in the function where the innermost container was incorrectly used when doing the insertion of the content.  Go layout tests.

[Dave Hyatt]

Created attachment 14284 [details]
Fix issues with table display types

Even though layout tests passed (and the reduction passed), my own further testing before landing caught some additional issues with table display types and generated content.  We weren't making the right kind of anonymous container for generated content even though we kept the table display type.

[Darin Adler]

Comment on attachment 14284 [details]
Fix issues with table display types

+    bool oldContentPresent = child;

For MSVC that needs to be !!child instead of just child used as a boolean.

r=me

[Dave Hyatt]

Fixed.