Bug 15736 – REGRESSION (r19695-r19697): Overlapping table content on mtp.msnbc.com (Quirksmode)

• Status: RESOLVED
• Resolution: FIXED
• Component: Layout and Rendering

[David Kilzer (:ddkilzer)]

* SUMMARY
The "TAKE TWO: msnbc.com" column in the right-center of the page has overlapping content in recent WebKit ToT builds.

* STEPS TO REPRODUCE
1. Launch Safari/WebKit.
2. Go to URL:  http://www.msnbc.msn.com/id/3032608/
3. Look at "TAKE TWO: msnbc.com" column.

* RESULTS
The first item in the list overlaps the header image (see soon-to-be-attached screenshot).

* REGRESSION:
This is a regression from Safari 2.0.4 (419.3) with original WebKit on Mac OS X 10.4.10 (8R218).  Tested with a local debug build of WebKit r27112 with Safari 3 Public Beta v. 3.0.3 (522.12.1) on Mac OS X 10.4.10 (8R218).

[David Kilzer (:ddkilzer)]

Created attachment 16915 [details]
Screenshot

[David Kilzer (:ddkilzer)]

Works: r19672  Fails: r19697

[David Kilzer (:ddkilzer)]

Checking internal builds:
Works: r19695  Fails: r19698

[David Kilzer (:ddkilzer)]

Almost certainly caused by r19696 as r19697 didn't change any table code:

http://trac.webkit.org/projects/webkit/changeset/19696

[David Kilzer (:ddkilzer)]

Created attachment 16926 [details]
Partial reduction (depends on external resources)

A partial reduction (still depends on external resources).

TOT matches Firefox on the reduction.

[David Kilzer (:ddkilzer)]

(In reply to comment #6)
> TOT matches Firefox on the reduction.

Both Firefox 2.0.0.8 and Firefox 3.0a9pre (Oct 28, 2007) render the same was as ToT WebKit on Tiger.

Opera 9.22 renders like Safari 2.0.4 (419.3) on Tiger, as does MSIE 7 on Windows XP Pro.

[David Kilzer (:ddkilzer)]

Created attachment 17258 [details]
Reduction #1

This reduction is truer to the original page.

Works in Opera 9.22 and original Safari 2.0.4, but not in Firefox 2.0.0.9 or ToT WebKit + Safari 3 Public Beta.  On Tiger.

[David Kilzer (:ddkilzer)]

Created attachment 17259 [details]
Reduction #2

This reduction does not include a table, so it is not as true to the original page.

Renders as expected on Opera 9.22.  Overlap occurs on Safari 2.0.4, ToT WebKit+Safari 3 Public Beta and Firefox 2.0.0.9 on Tiger.

[Robert Blaut]

David, Opera renders both test cases as expected due to Quirks Mode: http://www.opera.com/docs/specs/doctype/ It emulates IE bug: 

"Vertical overflow in block elements with a specified height

In Quirks Mode, Opera will expand the block element to fit the content. In Standards Mode, Opera honours the height and applies the style overflow:visible if nothing else is specified."

If you add proper DOCTYPE to your test cases Opera will start render it as Webkit, Gecko in standards mode.

So we want to emulate above mentioned Opera behavior as well it should be added to Quirks Mode of Webkit. If we consider this bug in CSS standard way the bug is INVALID.

[Robert Blaut]

The overflow:visible issue is covered also on this page: http://www.quirksmode.org/css/quirksmode.html#t16

[David Kilzer (:ddkilzer)]

Thanks for the detective work, Robert!

[Robert Blaut]

The issue doesn't block bug 9610 which only tracks problems reported on this page: http://www.quirksmode.org/bugreports/archives/safari/

[David Kilzer (:ddkilzer)]

(In reply to comment #13)
> The issue doesn't block bug 9610 which only tracks problems reported on this
> page: http://www.quirksmode.org/bugreports/archives/safari/

Bug 9610 was a blocker due to Comment #10 and Comment #11.

[David Kilzer (:ddkilzer)]

(In reply to comment #14)
> (In reply to comment #13)
> > The issue doesn't block bug 9610 which only tracks problems reported on this
> > page: http://www.quirksmode.org/bugreports/archives/safari/
> 
> Bug 9610 was a blocker due to Comment #10 and Comment #11.

Sorry, I misunderstood you Robert!

[Robert Hogan]

This looks good now - closing..