Summary: | table-layout: fixed overflow not painted in normal order | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | nemo <bugs> | ||||||
Component: | CSS | Assignee: | Dave Hyatt <hyatt> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | hyatt | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | All | ||||||||
OS: | OS X 10.5 | ||||||||
URL: | http://m8y.org/tmp/testcase103.xhtml | ||||||||
Attachments: |
|
Description
nemo
2009-02-04 14:01:24 PST
Created attachment 27900 [details]
Patch to fix stacking order issue
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.
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. Created attachment 27940 [details]
Patch
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.
|