Bug 14614

Summary:	REGRESSION (r14597): Odd spacing at marware.com
Product:	WebKit	Reporter:	Charlie Burtoff <cburtoff>
Component:	Evangelism	Assignee:	Nobody <webkit-unassigned>
Status:	RESOLVED FIXED
Severity:	Normal	CC:	markmalone, mitz, sstigler1985, webkit
Priority:	P2	Keywords:	Regression
Version:	523.x (Safari 3)
Hardware:	Mac
OS:	OS X 10.4
URL:	http://www.marware.com/PRODUCTS/iPhone-Cases

Description Charlie Burtoff 2007-07-13 14:10:38 PDT

Hello,

	I apologize in advance if I'm filling this bug incorrectly or to the wrong group. 

http://www.marware.com/PRODUCTS/iPhone-Cases

When I visit the URL above in FF or Safari in tiger the spacing between the products is correct, when I use webkit or my iphone (for what its worth) the spacing on the items listed on that page increase. 

Once again, I apologize if this is being filled handled incorrectly; sorry.

Comment 1 Alexey Proskuryakov 2007-07-14 02:12:25 PDT

Confirmed as a regression with r24241, thank you for reporting this!

One needs to scroll the page down to see the problem.

Comment 2 mitz 2007-07-14 03:46:42 PDT

The site sends different HTML to Safari than it sends to Firefox or curl. For example, curl sees

<tr valign='top' height=85>

where Safari gets

<tr valign='top' height=595>

Haven't checked with old Safari yet.

Comment 3 mitz 2007-07-14 04:25:53 PDT

Regressed in <http://trac.webkit.org/projects/webkit/changeset/14597>, fix for bug 3297. The website seems to employ a Safari-specific workaround for bug 3297. They should probably change the scope of the workaround.

Comment 4 Alexey Proskuryakov 2007-07-14 06:32:16 PDT

Mitz has discovered that one needs to trash marware.com cookies in addition to spoofing User Agent in order to get a Firefox version of the page, which works correctly. Moving to evangelism.

Comment 5 David Kilzer (:ddkilzer) 2007-07-14 16:15:15 PDT

I just sent an email to info@marware.com about the need to remove this workaround on their web site.

Comment 6 David Kilzer (:ddkilzer) 2007-07-14 16:16:25 PDT

(In reply to comment #5)
> I just sent an email to info@marware.com about the need to remove this
> workaround on their web site.

And a customer support case was immediately created!

Case # SUP107530 Created: Web site contains Safari-specific workaround which needs to be removed

Comment 7 Sam Stigler 2007-11-18 22:11:02 PST

The original URL given is now a 404 redirect to marware.com; the correct URL is http://www.marware.com/PRODUCTS/iPhone-Products .

Comment 8 David Kilzer (:ddkilzer) 2007-11-18 22:51:14 PST

(In reply to comment #6)
> (In reply to comment #5)
> > I just sent an email to info@marware.com about the need to remove this
> > workaround on their web site.
> 
> And a customer support case was immediately created!
> 
> Case # SUP107530 Created: Web site contains Safari-specific workaround which
> needs to be removed

I haven't heard anything since this was created, although I haven't tried to follow-up either.

Comment 9 Mark Malone 2007-11-19 10:21:28 PST

I'll ping the Apple folks with the Marware relationship.

Comment 10 Robert Blaut 2008-02-10 23:05:58 PST

It seems the bug is fixed. No more huge space between products. Any confirmations?

Comment 11 Robert Blaut 2008-07-17 01:09:44 PDT

(In reply to comment #10)
> It seems the bug is fixed. No more huge space between products. Any
> confirmations?
> 

As I said a half year ago the page renders correctly now in the latest Webkit. Closing as fixed.