-
Notifications
You must be signed in to change notification settings - Fork 113
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Respect self-closing tags only on foreign elements #137
Conversation
test/HTML5/Parser/TokenizerTest.php
Outdated
$this->assertEventEquals('startTag', $expects, $events->get(0)); | ||
$this->assertEventEquals('endTag', $expects, $events->get(1)); | ||
$this->assertTrue($events->get(0)['data'][2]); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This syntax fails in PHP 5.3
https://travis-ci.org/Masterminds/html5-php/jobs/311341191#L260
Also, any chance this will ever get merged?
Dac, Thanks for the review. It's fixed in my latest push, I think. Probably won't be merged as this repo seems pretty unmaintained. --Albert |
A stable project with really few bugs should not be defined as "unmaintained" 😃 |
Out-of-spec behaviour is not a bug, then? |
I'm not saying is it not a bug... just worried about BC breaks. From what I know Drupal guys could have a good test suite to check if this breaks something.. |
@xjm Is there anybody in drupal that can test this ? |
Tested, google chrome has the same behavior as this proposal. |
This fixes #136.
See also: https://www.w3.org/TR/html5/syntax.html#start-tags