Update to latest official #4
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Control of PDPage creation via Page supplier.
I'm the worst offender with tabs but they are evil!
Currently broken image links are causing a NPE.
Such as images.
Upgrade Jsoup dependency for last release of this sub-module. Users should use W3CDom class from Jsoup instead.
…ents. [ci skip]
If we advertise it, we should test it!
…f lines. Thanks @halcsi! Looks much better!
…k fonts are used. With tests. Thanks for reporting @daliuss!
This work moves sizing of replaced objects form externally in each replaced object implementation to internally in the core code. This should reduce duplication and mean all sizing properties are available on each replaced object type. For example, now images have min-width and min-height as well as a border-box implementation. This work is published initially in a new branch because while there are good tests for images we need to test the new sizing code for all the other replaced object options such as SVG, PDF, custom, etc.
Including min-width/min-height and border-box sizing. With test.
We can now use unit values such as mm in the width/height attribute of SVG images. With tests.
… default. Added new constructor so user can allow scripts and resource requests if using trusted SVGs only. With tests. Needs documenting in Wiki upon release.
Now, always keep the correct aspect ratio.
The MathML renderer (JEuclid) produces slightly different results on JDK8 vs JDK11 so is not suitable for auto testing.
Upgrade PDFBox to 2.0.16 and PDFBox-Graphics2d to 0.24
Only one broken test - svg linked with img tag.
…mg tag. This is mostly a revert of d9cdd6a I forgot about external SVGs rather than inline SVGs where the width/height attribute are converted to CSS earlier in the render process.
Nearly there, the only reason I havent committed the test is that it is a couple of pixels off on width, which is noticeable when there is a border.
Turns out the problem was prematurely casting our width and height to ints and thus getting incorrect scale factors. Test now passing.
I thought I had broken the clear mechanism, but it was fine. Left test as a useful regression test.
Hopefully this will also trigger travis.
Replaced element improved sizing support
Time to get rid of some of our 1000+ warnings.
+ Test with simple text only (passing). + Test with nested elements (failing). + Test with nested floats (failing).
So it can be worked with easier.
Better flowing column support
Was broken when intrinsic width or height were greater than max-width or max-height. Thanks again @rototor. Public domain images provided by pexabay at www.pexels.com Co-Authored-By: Emmeran Seehuber <rototor@rototor.de>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.