> This PR ensures the `tickMarksUpdated()` method (overriden with minor tick > mark layout code in subclasses) is called during `Chart` `Axis` layout when > needed. > Previously, it was only called when tick mark length or range was changed, > which is not happening during axis animations, causing minor ticks to become > outdated when animations are enabled. > > I've added a test that fails with the previous code, specifically checking > the tick mark spacing. > Alternatively the test could be implemented by accessing > ValueAxis.minorTickMarkValues, which would need to be made at least > package-private however (with a shim). This implementation checks the tick > mark `Path` shape instead, which is already publicly accessible (but could be > considered an implementation detail).
Markus Mack has updated the pull request incrementally with one additional commit since the last revision: use JUnit4 assertEquals ------------- Changes: - all: https://git.openjdk.org/jfx/pull/1489/files - new: https://git.openjdk.org/jfx/pull/1489/files/1a853794..81b4be6a Webrevs: - full: https://webrevs.openjdk.org/?repo=jfx&pr=1489&range=02 - incr: https://webrevs.openjdk.org/?repo=jfx&pr=1489&range=01-02 Stats: 3 lines in 1 file changed: 0 ins; 1 del; 2 mod Patch: https://git.openjdk.org/jfx/pull/1489.diff Fetch: git fetch https://git.openjdk.org/jfx.git pull/1489/head:pull/1489 PR: https://git.openjdk.org/jfx/pull/1489
