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)

x86
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: annevk, Assigned: dbaron)

Details

Attachments

(1 file)

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.
"Now"?  Did something change in the spec or in Firefox?
CSS 2.1 changed and http://dev.w3.org/csswg/css3-namespace/ changed accordingly. So not your fault :-)
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.
Flags: blocking1.9?
Attached patch patchSplinter Review
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)
Note that this fix essentially reverts bug 35813.
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+
Comment on attachment 302045 [details] [diff] [review]
patch

Simple fix to match a recent spec change.
Attachment #302045 - Flags: approval1.9?
Attachment #302045 - Flags: approval1.9? → approval1.9+
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.

Attachment

General

Created:
Updated:
Size: