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

Custom object drawers #78

Merged
merged 3 commits into from
Mar 16, 2017

Conversation

rototor
Copy link
Contributor

@rototor rototor commented Mar 12, 2017

Implement ObjectDrawer for <object> tags.

Implement a default factory for FSObjectDrawer, which allows to register custom drawers.

The testrunner includes a simple example how to implement a FSObjectDrawer.

Currently its only possible with the PdfBoxRendererBuilder to register a FSObjectDrawerFactory. As soon as #77 is merged I will implement the same for Java2D.

This is the final feature I need to migrate all my reports to OpenHtmlToPdf, as we do some custom graphs in some reports.

I hope you don't mind the auto organize of imports, which IDEA does all the time ... Otherwise I will try to minimize the import changes.

rototor added 3 commits March 12, 2017 19:32
Implement a default factory for FSObjectDrawer, which allows to
register custom drawers.

The testrunner includes a simple example how to implement
a FSObjectDrawer.

Currently its only possible with the PdfBoxRendererBuilder to register
a FSObjectDrawerFactory. As soon as danfickle#77 is merged I will implement
the same for Java2D.

This is the final feature I need to migrate all my reports
to OpenHtmlToPdf, as we do some custom graphs in some reports.
…ct-drawers

# Conflicts:
#
openhtmltopdf-core/src/main/java/com/openhtmltopdf/simple/Java2DRenderer
Builder.java
@danfickle danfickle merged commit f0bbb8d into danfickle:open-dev-v1 Mar 16, 2017
@rototor rototor mentioned this pull request Dec 24, 2017
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.

2 participants