HTML parser: <dialog> does not close <p> |
||||||||
Issue descriptionChrome Version: 59.0.3046.0 (Official Build) canary (64-bit) Test: http://software.hixie.ch/utilities/js/live-dom-viewer/saved/4976 Result in Chrome canary: log: <dialog><p></dialog>x log: <p><dialog> log: DONE Expect: log: DONE The HTML parser does not close the p for </dialog> in the first test, and doesn't close the p for <dialog> in the second test. HTML standard: [[ A start tag whose tag name is one of: "address", "article", "aside", "blockquote", "center", "details", "dialog", "dir", "div", "dl", "fieldset", "figcaption", "figure", "footer", "header", "hgroup", "main", "menu", "nav", "ol", "p", "section", "summary", "ul" If the stack of open elements has a p element in button scope, then close a p element. Insert an HTML element for the token. ]] [[ An end tag whose tag name is one of: "address", "article", "aside", "blockquote", "button", "center", "details", "dialog", "dir", "div", "dl", "fieldset", "figcaption", "figure", "footer", "header", "hgroup", "listing", "main", "menu", "nav", "ol", "pre", "section", "summary", "ul" [...handling of end tag...] ]] html5lib-tests PR: https://github.com/html5lib/html5lib-tests/pull/91 Test passes in Firefox Nightly. Fails in Safari TP.
,
Apr 5 2017
,
Apr 6 2018
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue. Sorry for the inconvenience if the bug really should have been left as Available. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Apr 9 2018
,
Oct 9
Can I take a look this issue if no one works on it?
,
Oct 9
> Can I take a look this issue if no one works on it? Please take this. But I'm afraid this issue isn't easy to fix. I guess we need to update html5lib in WPT before fixing Chrome code.
,
Oct 10
,
Oct 11
Fixing chrome code to solve this issue is easy but since html5lib should be migrated into WPT test first. Not easy for me right now.
,
Oct 11
|
||||||||
►
Sign in to add a comment |
||||||||
Comment 1 by dk...@chromium.org
, Apr 5 2017