Closed
Bug 323229
Opened 19 years ago
Closed 15 years ago
</head> strips spaces following it
Categories
(Core :: DOM: HTML Parser, defect, P4)
Tracking
()
RESOLVED
FIXED
Future
People
(Reporter: ian, Unassigned)
References
()
Details
(Whiteboard: [fixed by the HTML5 parser])
Attachments
(2 files)
|
2.33 KB,
patch
|
Details | Diff | Splinter Review | |
|
3.94 KB,
patch
|
Details | Diff | Splinter Review |
The following:
<pre>Hello</head> World</pre>
...renders with no spaces. In other browsers the spaces render.
TESTCASE: http://www.hixie.ch/tests/adhoc/html/parsing/019.html
SPECIFICATION: I haven't quite worked out the spec yet...
Note that in "</head> <body>" all browsers _do_ strip the spaces.
|
||
Comment 1•19 years ago
|
||
Easy to fix, but only 'cause I like you, Ian ;-).
Status: NEW → ASSIGNED
Priority: -- → P4
Whiteboard: [patch]
Target Milestone: --- → mozilla1.9alpha
|
|
||
Comment 2•19 years ago
|
||
Given the document: |<html><head><style>body { white-space: pre }</style></head> foo| you'll find us stripping whitespace, but I hope that's compatible (and that browsers don't find an implicit <body> in those spaces).
Attachment #208309 -
Flags: superreview?(jst)
Attachment #208309 -
Flags: review?(jst)
|
|
||
Comment 3•19 years ago
|
||
dbaron suggested this. I have no real feelings one way or the other...
Attachment #208313 -
Flags: superreview?(jst)
Attachment #208313 -
Flags: review?(jst)
|
|
||
Comment 4•19 years ago
|
||
Comment on attachment 208309 [details] [diff] [review]
Only strip ws before the body or frameset
Clearing review requests pending a decision.
Attachment #208309 -
Flags: superreview?(jst)
Attachment #208309 -
Flags: review?(jst)
|
|
||
Updated•19 years ago
|
Attachment #208313 -
Flags: superreview?(jst)
Attachment #208313 -
Flags: review?(jst)
|
|
||
Comment 5•19 years ago
|
||
When there's been a concensus, I'll look at this again.
Target Milestone: mozilla1.9alpha → Future
|
||
Comment 6•19 years ago
|
||
So what does the second patch actually do? As things stand, someone would have to review both patches to be able to say anything intelligent about them.
|
|
||
Comment 7•19 years ago
|
||
It doesn't strip spaces following any tokens at all. I'm really waiting on the spec to finish changing (since that's whawt Hixie told me to do when he filed this).
|
|
||
Comment 8•19 years ago
|
||
Ah, ok.
|
|
||
Updated•16 years ago
|
Assignee: mrbkap → nobody
|
||
Comment 9•15 years ago
|
||
Hey, the HTML5 parser fixed a bug filed Hixie. :-)
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Whiteboard: [patch] → [fixed by the HTML5 parser]
You need to log in
before you can comment on or make changes to this bug.
Description
•