Closed Bug 35813 Opened 25 years ago Closed 24 years ago

case sensitivity of CSS namespaces code [SELECT]

Categories

(Core :: CSS Parsing and Computation, defect, P2)

defect

Tracking

()

VERIFIED FIXED

People

(Reporter: dbaron, Assigned: dbaron)

Details

(Keywords: css3, Whiteboard: [fix in hand][nsbeta3+])

Attachments

(4 files)

According to http://www.w3.org/TR/REC-CSS2/syndata.html#q4 , nothing under the control of CSS is case-sensitive. Therefore, namespace prefixes in CSS should not be case sensitive. I have not tested this, but the code makes it look like we do this wrong. I should probably bring this up on www-style too... people might want namespaces to be case-senstitive, although it's unusual for CSS. See also bug 12450 (same area of the code).
Yes, this is definitely a bug. Attaching one simple testcase. I think I can probably fix it, so I'm holding on to it.
Attached file stylesheet for testcase (deleted) —
Attached file one simple testcase (deleted) —
Accepting, M18. I want to fix this after beta2 goes out. (I don't think it's worth introducing risk at this point.) (Unless, pierre, you want to fix this sooner.)
Status: NEW → ASSIGNED
Target Milestone: --- → M18
Keywords: css3
Target Milestone: M18 → M19
As per meeting with ChrisD yesterday, taking QA. Nominating for nsbeta3. This is a standards compliance issue.
Keywords: correctness, nsbeta3
QA Contact: chrisd → py8ieh=bugzilla
Summary: case sensitivity of CSS namespaces code → case sensitivity of CSS namespaces code [SELECT]
Target Milestone: M19 → M18
Priority: P3 → P2
Whiteboard: [fix in hand]
Marking nsbeta3+ per ekrock's email.
Whiteboard: [fix in hand] → [fix in hand][nsbeta3+]
Fix checked in 2000-09-07 19:05.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Testcase is invalid, will attach new testcase.
Well, I get a crash from the new stylesheet too. Oh well. New testcase coming up for completeness (uses new stylesheet). Note - these crashes are unrelated to this bug, since I am using a build that was made after the fix went in.
Attached file New, simpler, testcase. (deleted) —
VERIFIED FIXED on Windows 2000 Commercial Build 2000091308 and Linux Commercial Build 2000091312 using second testcase.
Status: RESOLVED → VERIFIED
This fix is being reverted in bug 416106.
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: