Closed
Bug 569528
Opened 15 years ago
Closed 15 years ago
[HTML5] <h5> inside <button> inside <p> implicitly closes the <p>
Categories
(Core :: DOM: HTML Parser, defect, P2)
Core
DOM: HTML Parser
Tracking
()
RESOLVED
FIXED
mozilla2.0b7
| Tracking | Status | |
|---|---|---|
| blocking2.0 | --- | final+ |
People
(Reporter: justin.lebar+bug, Assigned: hsivonen)
References
Details
Attachments
(2 files, 1 obsolete file)
|
60 bytes,
text/html
|
Details | |
|
12.65 KB,
patch
|
sicking
:
review+
|
Details | Diff | Splinter Review |
Originally filed as https://bugs.launchpad.net/bugs/588595
The following code:
<p> <button> <h5>Bar</h5> </button> </p>
results in the following DOM tree:
<p> <button> </button></p><h5>Bar</h5> <p></p>
which, of course, renders incorrectly.
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.3a5pre) Gecko/20100601 Minefield/3.7a5pre
|
||
Updated•15 years ago
|
See Also: → https://launchpad.net/bugs/588595
|
|
||
Comment 1•15 years ago
|
||
I'm affected on Ubuntu as well, so this seems to be platform independent:
Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.3a5pre) Gecko/20100601 Ubuntu/10.04 (lucid) Minefield/3.7a5pre
OS: Windows XP → All
Hardware: x86 → All
|
Assignee | |
Comment 2•15 years ago
|
||
The test case isn't really a test case for this bug. Attaching a test case.
The behavior (and the spec) here changed at bug 558302 in an effort to match WebKit and IE8 more closely. It appears that spec and parser change didn't quite capture all important aspects of WebKit and IE8 behavior.
|
|
Assignee | |
Comment 3•15 years ago
|
||
Spec bug filed http://www.w3.org/Bugs/Public/show_bug.cgi?id=9829
Priority: -- → P2
|
|
Assignee | |
Updated•15 years ago
|
Summary: [HTML5] <h5> inside <button> inside <p> handled incorrectly → [HTML5] <h5> inside <button> inside <p> implicitly closes the <p>
|
|
||
Updated•15 years ago
|
blocking2.0: --- → ?
|
||
Updated•15 years ago
|
blocking2.0: ? → final+
|
Reporter | |
Comment 6•15 years ago
|
||
Henri, is this invalid HTML?
|
|
Assignee | |
Comment 7•15 years ago
|
||
(In reply to comment #6)
> Henri, is this invalid HTML?
Not per the current draft, I believe.
It's clear that the parser needs changing. I'll get to it.
|
|
Assignee | |
Comment 8•15 years ago
|
||
This patch doesn't have a test case, because bug 559023 will add more than enough <button> test cases.
Attachment #473095 -
Flags: review?(jonas)
Comment on attachment 473095 [details] [diff] [review]
Fix implementing spec revision 5158
rs=me
Attachment #473095 -
Flags: review?(jonas) → review+
|
|
Assignee | |
Comment 10•15 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
|
|
Assignee | |
Comment 11•15 years ago
|
||
Marking as in-testsuite?, since the test cases will land as part of bug 559023.
Flags: in-testsuite?
|
|
Assignee | |
Comment 12•15 years ago
|
||
Test landed as part of bug 559023.
Flags: in-testsuite? → in-testsuite+
Target Milestone: --- → mozilla2.0b7
You need to log in
before you can comment on or make changes to this bug.
Description
•