Bug 23740 - table-layout: fixed overflow not painted in normal order
Summary: table-layout: fixed overflow not painted in normal order
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: CSS (show other bugs)
Version: 528+ (Nightly build)
Hardware: All OS X 10.5
: P2 Normal
Assignee: Dave Hyatt
URL: http://m8y.org/tmp/testcase103.xhtml
Keywords:
Depends on:
Blocks:
 
Reported: 2009-02-04 14:01 PST by nemo
Modified: 2009-02-24 17:31 PST (History)
1 user (show)

See Also:


Attachments
Patch to fix stacking order issue (38.07 KB, patch)
2009-02-23 18:41 PST, Dave Hyatt
no flags Details | Formatted Diff | Diff
Patch (40.94 KB, patch)
2009-02-24 15:56 PST, Dave Hyatt
eric: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description nemo 2009-02-04 14:01:24 PST
As described in http://m8y.org/tmp/testcase103.xhtml, after initially blaming Webkit, I decided that Firefox' behaviour, even though it had been as I had initially expected, was inconsistent.

However, after bz pointed out the relevant portion of the spec on painting order, I understood that, in fact, Webkit appears to be ignoring the painting of backgrounds first, then text.

And that, in fact, I've come to expect that behaviour throughout the web when content overflows.

Furthermore, bz pointed out that normally webkit does the right thing.

He offered the following data URI as a demonstration.
data:text/html,<div style="background: red; color: white; height: 100px; width: 200px">Text</div><div style="margin-top: -100px; background: green; width: 100px; height: 100px"></div>

The workaround is to specify an explicit z-index for webkit, which fortunately I can do in this case fairly easily.
Comment 1 Dave Hyatt 2009-02-23 18:41:45 PST
Created attachment 27900 [details]
Patch to fix stacking order issue
Comment 2 Eric Seidel (no email) 2009-02-24 12:55:31 PST
Comment on attachment 27900 [details]
Patch to fix stacking order issue

Hyatt and I talked about this a lot over IRC.  He's gonna update the patch and then I'll make an official review.
Comment 3 nemo 2009-02-24 15:03:42 PST
Sweet. Thanks.
I thought initially this was going to end up a WONTFIX.

Nice to not have to worry about it.  It did suck up an afternoon figuring out what the heck was going on.
Comment 4 Dave Hyatt 2009-02-24 15:56:26 PST
Created attachment 27940 [details]
Patch
Comment 5 Eric Seidel (no email) 2009-02-24 16:01:35 PST
Comment on attachment 27940 [details]
Patch

The changelog doens't mention your isMask() changes.  your test cases are missing newlines at the end.  you added some spaces after shouldBeNormalFlowOnly which you didn't mean to.

Otherwise this looks fine, as far as I can tell.  You'll need a ChangeLog for the LayoutTests directory as well.
Comment 6 Dave Hyatt 2009-02-24 17:31:23 PST
Fixed in r41203.