Bug 17564

Summary: REGRESSION(r29976): discovermagazine.com lays out incorrect
Product: WebKit Reporter: Michelle <maddietrich>
Component: Layout and RenderingAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: hyatt, mitz, mrowe
Priority: P1 Keywords: HasReduction, InRadar, Regression
Version: 528+ (Nightly build)   
Hardware: Mac   
OS: OS X 10.5   
URL: http://discovermagazine.com
Attachments:
Description Flags
Reduction none

Michelle
Reported 2008-02-26 17:34:08 PST
Malformed
Attachments
Reduction (280 bytes, text/html)
2008-02-26 20:00 PST, Mark Rowe (bdash) 		no flags
Details
View All Add attachment proposed patch, testcase, etc.
Mark Rowe (bdash)
Comment 1 2008-02-26 18:14:25 PST
The page lays out incorrectly in TOT. It lays out correctly in Safari 3.0.4.
Mark Rowe (bdash)
Comment 2 2008-02-26 18:14:39 PST
<rdar://problem/5767534>
Mark Rowe (bdash)
Comment 3 2008-02-26 19:18:11 PST
This was introduced by the fix for bug 16751 in r29976.
mitz
Comment 4 2008-02-26 19:50:24 PST
There are no plans to revert that fix.
Mark Rowe (bdash)
Comment 5 2008-02-26 19:53:06 PST
They don't appear to be using the "star seven" hack. We're tripping over this rule: .mLeft form div img., .mLeft form div label { margin-left: 2px; float: left; } Note the extra "." before the ",".
Dave Hyatt
Comment 6 2008-02-26 19:59:33 PST
The site is 100% broken for me in Safari 3, so I do not believe the star seven hack is to blame.
Dave Hyatt
Comment 7 2008-02-26 20:00:01 PST
This page does *not* lay out correctly in Safari 3.0.4 for me. It is wrong 100% of the time and looks identical to ToT.
Mark Rowe (bdash)
Comment 8 2008-02-26 20:00:46 PST
Created attachment 19393 [details] Reduction
Mark Rowe (bdash)
Comment 9 2008-11-30 10:14:37 PST
This was fixed with <http://trac.webkit.org/changeset/35403>.
Note You need to log in before you can comment on or make changes to this bug.