Skip to content

Add tests for <plaintext> in all insertion modes + in svg #87

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Feb 3, 2017
Merged

Add tests for <plaintext> in all insertion modes + in svg #87

merged 1 commit into from
Feb 3, 2017

Conversation

zcorpan
Copy link
Contributor

@zcorpan zcorpan commented Jan 12, 2017

@zcorpan zcorpan requested a review from gsnedders January 12, 2017 00:17
@zcorpan zcorpan mentioned this pull request Jan 12, 2017
Closed
@zcorpan
Copy link
Contributor Author

zcorpan commented Jan 12, 2017

@sideshowbarker @hsivonen FYI /parsetree/ doesn't produce a body element for at least one test case added here.

sideshowbarker added a commit to validator/validator that referenced this pull request Jan 12, 2017
@sideshowbarker
Fixed regression in <template> parsing
98aafd9 
See html5lib/html5lib-tests#87 (comment)
Thanks @zcorpan
@sideshowbarker
Copy link
Contributor

@sideshowbarker @hsivonen FYI /parsetree/ doesn't produce a body element for at least one test case added here.

Thanks for catching this. Fixed now in the https://github.com/validator/htmlparser sources and pushed to https://checker.html5.org/

@zcorpan
Copy link
Contributor Author

zcorpan commented Feb 3, 2017

Ping 😊

@gsnedders gsnedders merged commit 5fca014 into master Feb 3, 2017
@gsnedders gsnedders deleted the plaintext branch February 3, 2017 18:29
@gsnedders gsnedders mentioned this pull request Mar 15, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gsnedders gsnedders Awaiting requested review from gsnedders

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@zcorpan @sideshowbarker @gsnedders