Skip to content
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

Introduce DefaultHtmlBuilder #196

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

obecker
Copy link
Contributor

@obecker obecker commented Jun 19, 2021

to have a central and easy to use entry point for configuring the HTML
rendering, and to hide the details of FlatHtml and IndentedHtml.

Adds static methods in Indenter for easy changing the indent depth.

This commits also adds @deprecated annotations to some methods and
fields that should be removed in a future version of the library, namely

  • some static fields of Config
  • the public visibility of FlatHtml and IndentedHtml

Resolves #193

to have a central and easy to use entry point for configuring the HTML
rendering, and to hide the details of FlatHtml and IndentedHtml.

Adds static methods in Indenter for easy changing the indent depth.

This commits also adds @deprecated annotations to some methods and
fields that should be removed in a future version of the library, namely
 - some static fields of Config
 - the public visibility of FlatHtml and IndentedHtml
@obecker
Copy link
Contributor Author

obecker commented Jun 19, 2021

@sembler by the way: feel free to suggest a better name, since DefaultHtmlBuilder does not implement the HtmlBuilder interface

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.

render API simplification
1 participant