Looking for saved searches? click on "Search Bugs" above.
[HTML5] <h5> inside <button> inside <p> implicitly closes the <p>
RESOLVED
FIXED
in mozilla2.0b7
Status
()
People
(Reporter: Justin Lebar (not reading bugmail), Assigned: hsivonen)
Tracking
Bug Flags:
Firefox Tracking Flags
(blocking2.0 final+)
Details
Attachments
(2 attachments, 1 obsolete attachment)
|
60 bytes,
text/html
|
Details | |
|
12.65 KB,
patch
|
sicking
:
review+
|
Details | Diff | Splinter Review |
Created attachment 448693 [details] Testcase 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•8 years ago
|
||
See Also: → https://launchpad.net/bugs/588595
|
|
||
Comment 1•8 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•8 years ago
|
||
Created attachment 448704 [details] Actual test case 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•8 years ago
|
||
Spec bug filed http://www.w3.org/Bugs/Public/show_bug.cgi?id=9829
Priority: -- → P2
|
|
(Assignee) | |
Updated•8 years ago
|
||
Summary: [HTML5] <h5> inside <button> inside <p> handled incorrectly → [HTML5] <h5> inside <button> inside <p> implicitly closes the <p>
blocking2.0: --- → ?
|
||
Updated•8 years ago
|
||
blocking2.0: ? → final+
|
(Reporter) | |
Comment 6•7 years ago
|
||
Henri, is this invalid HTML?
|
|
(Assignee) | |
Comment 7•7 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•7 years ago
|
||
Created attachment 473095 [details] [diff] [review] Fix implementing spec revision 5158 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•7 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/41aaa362225b
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
|
|
(Assignee) | |
Comment 11•7 years ago
|
||
Marking as in-testsuite?, since the test cases will land as part of bug 559023.
Flags: in-testsuite?
|
|
(Assignee) | |
Comment 12•7 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
•