WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
14614
REGRESSION (
r14597
): Odd spacing at marware.com
https://bugs.webkit.org/show_bug.cgi?id=14614
Summary
REGRESSION (r14597): Odd spacing at marware.com
Charlie Burtoff
Reported
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.
Attachments
Add attachment
proposed patch, testcase, etc.
Alexey Proskuryakov
Comment 1
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.
mitz
Comment 2
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.
mitz
Comment 3
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.
Alexey Proskuryakov
Comment 4
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.
David Kilzer (:ddkilzer)
Comment 5
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.
David Kilzer (:ddkilzer)
Comment 6
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
Sam Stigler
Comment 7
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
.
David Kilzer (:ddkilzer)
Comment 8
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.
Mark Malone
Comment 9
2007-11-19 10:21:28 PST
I'll ping the Apple folks with the Marware relationship.
Robert Blaut
Comment 10
2008-02-10 23:05:58 PST
It seems the bug is fixed. No more huge space between products. Any confirmations?
Robert Blaut
Comment 11
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.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug