Closed
Bug 591589
Opened 14 years ago
Closed 14 years ago
make nbsp an identifier character
Categories
(Core :: CSS Parsing and Computation, defect)
Core
CSS Parsing and Computation
Tracking
()
RESOLVED
FIXED
mozilla2.0b7
People
(Reporter: dbaron, Assigned: dbaron)
Details
Attachments
(1 file)
|
4.11 KB,
patch
|
bzbarsky
:
review+
roc
:
approval2.0+
|
Details | Diff | Splinter Review |
At the CSS working group meeting earlier this week, the working group resolved the inconsistency between the grammar and the prose regarding which characters could be in and start an identifier by compromising between the two. We previously followed the prose; other browsers previously followed the grammar.
This means we need to change non-breaking space (NBSP) into an ident-start character.
See http://wiki.csswg.org/spec/css2.1#issue-196
Attachment #470171 -
Flags: review?(bzbarsky)
|
||
Comment 1•14 years ago
|
||
Comment on attachment 470171 [details] [diff] [review]
patch
Looks fine, but can't you start the invalid identifier range with 0x7B in the test?
Attachment #470171 -
Flags: review?(bzbarsky) → review+
|
Assignee | |
Updated•14 years ago
|
Attachment #470171 -
Flags: approval2.0?
|
|
Assignee | |
Comment 2•14 years ago
|
||
(In reply to comment #1)
> Looks fine, but can't you start the invalid identifier range with 0x7B in the
> test?
Sure. I have it changed locally.
Attachment #470171 -
Flags: approval2.0? → approval2.0+
|
|
Assignee | |
Comment 3•14 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b7
You need to log in
before you can comment on or make changes to this bug.
Description
•