On Tue, 25 Jun 2024 21:46:25 GMT, Markus Mack <mm...@openjdk.org> wrote:
>> 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 two additional > commits since the last revision: > > - cleanup imports > - add clarifying comment to test Simple enough fix, I think 1 reviewer is sufficient, but feel free. Also tested with AreaChart, BubbleChart, ScatterChart, StackedAreaChart. ------------- Marked as reviewed by angorya (Reviewer). PR Review: https://git.openjdk.org/jfx/pull/1489#pullrequestreview-2139969721
