Closed
Bug 416106
Opened 16 years ago
Closed 16 years ago
author specified identifiers are now case-sensitive
Categories
(Core :: CSS Parsing and Computation, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: annevk, Assigned: dbaron)
Details
Attachments
(1 file)
7.38 KB,
patch
|
bzbarsky
:
review+
bzbarsky
:
superreview+
mtschrep
:
approval1.9+
|
Details | Diff | Splinter Review |
CSS author specifiied identifiers, such as CSS namespace prefixes, are now case-sensitive. @namespace foo "x"; FOO|x { background:red } will not match element x in the x namespace. Instead, the rule will be dropped.
Reporter | ||
Comment 1•16 years ago
|
||
Testcase: http://tc.labs.opera.com/css/namespaces/001.xml
Comment 2•16 years ago
|
||
"Now"? Did something change in the spec or in Firefox?
Reporter | ||
Comment 3•16 years ago
|
||
CSS 2.1 changed and http://dev.w3.org/csswg/css3-namespace/ changed accordingly. So not your fault :-)
Assignee | ||
Comment 4•16 years ago
|
||
Though it changed to match the behavior we've always had for counters, IIRC. I think we should get this change into Firefox 3 since we're better off breaking compatibility sooner rather than later.
Updated•16 years ago
|
Flags: blocking1.9?
Assignee | ||
Comment 5•16 years ago
|
||
Make namespace identifiers case sensitive, and add tests for both that and for counter names being case sensitive.
Assignee: nobody → dbaron
Status: NEW → ASSIGNED
Attachment #302045 -
Flags: superreview?(bzbarsky)
Attachment #302045 -
Flags: review?(bzbarsky)
Comment 7•16 years ago
|
||
Comment on attachment 302045 [details] [diff] [review] patch Looks great.
Attachment #302045 -
Flags: superreview?(bzbarsky)
Attachment #302045 -
Flags: superreview+
Attachment #302045 -
Flags: review?(bzbarsky)
Attachment #302045 -
Flags: review+
Assignee | ||
Comment 8•16 years ago
|
||
Comment on attachment 302045 [details] [diff] [review] patch Simple fix to match a recent spec change.
Attachment #302045 -
Flags: approval1.9?
Updated•16 years ago
|
Attachment #302045 -
Flags: approval1.9? → approval1.9+
Assignee | ||
Comment 9•16 years ago
|
||
Checked in to trunk, 2008-02-08 11:57 -0800.
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•