Skip to content

Conversation

@rototor
Copy link
Contributor

@rototor rototor commented Sep 4, 2017

This pull request does the following things:

  • Upgrade PDFBox to 2.0.7
  • Upgrade ICU to 59.1
  • Upgrade PDFBox-Graphics2D to 0.7
  • Try to use fonts when rendering text with drawWithGraphics().

Should I split the version changes into a separate pull request or is this ok?

as much as possible. To do this the font resolving is bridged to
the PdfBoxFontResolver. As Times-Roman is the default fallback, detect it
and don't use it if Times New Roman is not explicit requested.

Note: The generated PDF views fine in Mac OS X preview but has some
"unkown" error when displayed in Acrobat. I need to investigate that.
@danfickle
Copy link
Owner

Thanks a lot @rototor

@danfickle danfickle merged commit f623251 into danfickle:open-dev-v1 Sep 7, 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