Skip to content

feat: Nokogiri::HTML5::Builder#3119

Merged
flavorjones merged 1 commit intomainfrom
flavorjones-html5-builder
Feb 3, 2024
Merged

feat: Nokogiri::HTML5::Builder#3119
flavorjones merged 1 commit intomainfrom
flavorjones-html5-builder

Conversation

@flavorjones
Copy link
Copy Markdown
Member

What problem is this PR intended to solve?

Provide a Builder class that generates HTML5 documents.

The construction behavior is identical to HTML4::Builder, but HTML5 documents serialize according to the HTML5 standard (see xml_node.c:output_node()).

Have you included adequate test coverage?

Added tests are a copied-and-modified version of the HTML4::Builder tests.

Does this change affect the behavior of either the C or the Java implementations?

The Java implementation still does not support HTML5.

@flavorjones flavorjones force-pushed the flavorjones-html5-builder branch from 47a4d8e to 9d8caee Compare February 3, 2024 15:09
which behaves like HTML4::Builder but returns an HTML5::Document
@flavorjones flavorjones force-pushed the flavorjones-html5-builder branch from 9d8caee to 3cb06c5 Compare February 3, 2024 16:33
@flavorjones flavorjones merged commit 91daaa7 into main Feb 3, 2024
@flavorjones flavorjones deleted the flavorjones-html5-builder branch February 3, 2024 19:27
@flavorjones flavorjones added this to the v1.17.0 milestone Feb 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant