Bug 15260 - Any styles defined after <element> {<property>:%} style are ignored by Safari
Summary: Any styles defined after <element> {<property>:%} style are ignored by Safari
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: CSS (show other bugs)
Version: 523.x (Safari 3)
Hardware: All All
: P2 Normal
Assignee: Nobody
URL: http://classifieds.sulekha.com/
Keywords: HasReduction
Depends on:
Blocks:
 
Reported: 2007-09-22 22:05 PDT by Anantha Keesara
Modified: 2007-10-07 00:50 PDT (History)
2 users (show)

See Also:


Attachments
Any styles defined after <element> {<property>:%} style are ignored by Safari (221 bytes, text/html)
2007-09-22 22:07 PDT, Anantha Keesara
no flags Details
First attempt (4.65 KB, patch)
2007-09-23 06:54 PDT, Rob Buis
mjs: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Anantha Keesara 2007-09-22 22:05:54 PDT
URL:  http://classifieds.sulekha.com/

Issue:
Any styles defined after 'width:%' are ignored

Other browsers: IE, FF, Opera no issue.
Comment 1 Anantha Keesara 2007-09-22 22:07:03 PDT
Created attachment 16355 [details]
Any styles defined after <element> {<property>:%} style are ignored by Safari
Comment 2 Gavin Sherlock 2007-09-22 23:15:59 PDT
Looks bad in both release Safari and the current nightly, so is not a regression.
Comment 3 Rob Buis 2007-09-23 06:54:33 PDT
Created attachment 16360 [details]
First attempt

With this patch we simply ignore the declaration with the invalid length specification.
Cheers,

Rob.
Comment 4 mitz 2007-09-23 07:12:24 PDT
Comment on attachment 16360 [details]
First attempt

+        Test for <a href="http://bugs.webkit.org/show_bug.cgi?id=15260">http://bugs.webkit.org/show_bug.cgi?id=15260</a>Any styles defined after <element> {<property>:%} style are ignored by Safari

Please replace the <'s there with entities. I also think that a text-only test would be far better.
Comment 5 Maciej Stachowiak 2007-09-29 22:17:05 PDT
Comment on attachment 16360 [details]
First attempt

r=me for feature branch (or trunk when it reopens)
Comment 6 Rob Buis 2007-10-03 08:08:41 PDT
Landed in r26015 on fb.