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

Embedded fonts #130

Merged
merged 17 commits into from Jan 3, 2018
Merged

Embedded fonts #130

merged 17 commits into from Jan 3, 2018

Conversation

neoancient
Copy link
Member

Switches mech printing from svgSalamander to Apache's Batik. Embedded font support in svgSalamander is broken and fixing it would be non-trivial. The one advantage of svgSalamander is its small size, but Batik has better support and documentation and supports features not found in svgSalamander.

Currently both libraries are present, since non-mech printing is still using svgSalamander. Once the remaining record sheets are transitioned to the new approach we can remove the older library.

Added fill as xml attribute in text elements where batik was not using
the CSS value.
Changed font measuring to use FontMetrics instead of adding a new
element and measuring the bounding box.
There can be a problem rendering a PNG that I traced down to an array
index out of range while copying the raster data but could find no way
to head it off. So far it's just one image that I encountered this with.
@neoancient neoancient merged commit 6d8b80e into master Jan 3, 2018
@neoancient neoancient deleted the embedded_fonts branch January 3, 2018 04:05
neoancient added a commit that referenced this pull request Jan 3, 2018
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.

None yet

1 participant