> In https://github.com/openjdk/jfx/pull/1405, I identified some shortcomings > of the stub font implementation. As I don't want to clutter the PR with that, > I decided to cherrypick the improvements I did to a new ticket and PR. > > The current implementation has the following shortcomings: > - It does not reliable detect the System Font, as a consequence, tests in > TableColumnHeaderTest.java are failing on my local machine > - Another consequence of this is, that the font size is always estimated with > 0, as it is not detected > - One shortcoming currently is, that the stub font siie estimate is not > considering bold fonts. That would improve writing tests for some scenarios, > e.g. for TableColumnHeader, where we would expect that the size of the header > is bigger since it is bold > > Some tests were failing for the following reasons: > - `AreaChartTest.java` - `expected -30.0, was -30.00000000004` - I added > ceiling to the data. > - `StackedBarChartTest.java` - since we now calculate correctly, the path > changed > - A test tried to load `Helvetica`, which is not supported in the stub font > loader. I changed it > - The default System font is considered a `Regular` one (style) > > I wrote tests and documented the stub behaviour. > I did some minor changes here: > - System font is now detected, also in bold and italic > - A bold font will be calculated with a little bit more width (1px). Checkout > the test as well for that > > Note: This only changes test setup, no 'production' code.
Marius Hanl has updated the pull request incrementally with one additional commit since the last revision: improve (stub) font tests, fallback and documentation ------------- Changes: - all: https://git.openjdk.org/jfx/pull/1422/files - new: https://git.openjdk.org/jfx/pull/1422/files/ec1b2abe..f8909afc Webrevs: - full: https://webrevs.openjdk.org/?repo=jfx&pr=1422&range=01 - incr: https://webrevs.openjdk.org/?repo=jfx&pr=1422&range=00-01 Stats: 98 lines in 4 files changed: 36 ins; 8 del; 54 mod Patch: https://git.openjdk.org/jfx/pull/1422.diff Fetch: git fetch https://git.openjdk.org/jfx.git pull/1422/head:pull/1422 PR: https://git.openjdk.org/jfx/pull/1422
