Re: RFR: 8314754: Minor ticks are not getting updated both the axes in LineChart [v3]

2024-06-27 Thread duke
On Wed, 26 Jun 2024 06:33:43 GMT, Markus Mack 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 chang

Re: RFR: 8314754: Minor ticks are not getting updated both the axes in LineChart [v3]

2024-06-26 Thread Andy Goryachev
On Wed, 26 Jun 2024 06:33:43 GMT, Markus Mack 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 chang

Re: RFR: 8314754: Minor ticks are not getting updated both the axes in LineChart [v3]

2024-06-26 Thread Marius Hanl
On Wed, 26 Jun 2024 06:33:43 GMT, Markus Mack 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 chang

Re: RFR: 8314754: Minor ticks are not getting updated both the axes in LineChart [v3]

2024-06-25 Thread Markus Mack
> 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, causi

Re: RFR: 8314754: Minor ticks are not getting updated both the axes in LineChart [v2]

2024-06-25 Thread Markus Mack
On Tue, 25 Jun 2024 22:48:26 GMT, Marius Hanl wrote: >> Markus Mack has updated the pull request incrementally with two additional >> commits since the last revision: >> >> - cleanup imports >> - add clarifying comment to test > > modules/javafx.controls/src/test/java/test/javafx/scene/chart/

Re: RFR: 8314754: Minor ticks are not getting updated both the axes in LineChart [v2]

2024-06-25 Thread Marius Hanl
On Tue, 25 Jun 2024 21:46:25 GMT, Markus Mack 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 chang

Re: RFR: 8314754: Minor ticks are not getting updated both the axes in LineChart [v2]

2024-06-25 Thread Andy Goryachev
On Tue, 25 Jun 2024 22:13:34 GMT, Markus Mack wrote: >> but wait: this may not be a different issue. I see a slightly different >> behavior relative to the master branch: >> >> - select bubble chart page in the monkey tester >> - add series with duplicate category >> >> now I see the minor ti

Re: RFR: 8314754: Minor ticks are not getting updated both the axes in LineChart [v2]

2024-06-25 Thread Andy Goryachev
On Tue, 25 Jun 2024 21:46:25 GMT, Markus Mack 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 chang

Re: RFR: 8314754: Minor ticks are not getting updated both the axes in LineChart [v2]

2024-06-25 Thread Markus Mack
On Tue, 25 Jun 2024 21:51:36 GMT, Andy Goryachev wrote: >> I think we need to create that as it turned out it's not the same as the >> minor tick problem fixed by this PR. > > but wait: this may not be a different issue. I see a slightly different > behavior relative to the master branch: > >

Re: RFR: 8314754: Minor ticks are not getting updated both the axes in LineChart [v2]

2024-06-25 Thread Andy Goryachev
On Tue, 25 Jun 2024 21:41:54 GMT, Markus Mack wrote: >> modules/javafx.controls/src/test/java/test/javafx/scene/chart/XYNumberLineChartsTest.java >> line 156: >> >>> 154: NumberAxis yAxis = (NumberAxis)chart.getYAxis(); >>> 155: >>> 156: List majorTickYValues = >>> yAxis.getCh

Re: RFR: 8314754: Minor ticks are not getting updated both the axes in LineChart [v2]

2024-06-25 Thread Markus Mack
On Tue, 25 Jun 2024 21:43:58 GMT, Andy Goryachev wrote: >> yes, the BarChart issue (which I actually want to get fixed) will need >> additional fixes in other classes. I could add those to this PR, but >> probably it's best to handle them in another ticket+PR to keep this PR >> affecting all `

Re: RFR: 8314754: Minor ticks are not getting updated both the axes in LineChart [v2]

2024-06-25 Thread Andy Goryachev
On Tue, 25 Jun 2024 21:49:54 GMT, Markus Mack wrote: >> a separate PR would be best. I am getting lost - do we have a ticket for >> that issue already or do we need to create one? > > I think we need to create that as it turned out it's not the same as the > minor tick problem fixed by this PR

Re: RFR: 8314754: Minor ticks are not getting updated both the axes in LineChart [v2]

2024-06-25 Thread Andy Goryachev
On Tue, 25 Jun 2024 21:42:23 GMT, Markus Mack wrote: >> modules/javafx.controls/src/main/java/javafx/scene/chart/Axis.java line 753: >> >>> 751: >>> 752: // call tick marks updated to inform subclasses that we >>> have updated tick marks >>> 753: tickMarksUpdated(); >>

Re: RFR: 8314754: Minor ticks are not getting updated both the axes in LineChart [v2]

2024-06-25 Thread Markus Mack
> 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, causi

Re: RFR: 8314754: Minor ticks are not getting updated both the axes in LineChart [v2]

2024-06-25 Thread Markus Mack
On Tue, 25 Jun 2024 20:06:29 GMT, Andy Goryachev wrote: >> Markus Mack has updated the pull request incrementally with two additional >> commits since the last revision: >> >> - cleanup imports >> - add clarifying comment to test > > modules/javafx.controls/src/main/java/javafx/scene/chart/Ax

Re: RFR: 8314754: Minor ticks are not getting updated both the axes in LineChart

2024-06-25 Thread Andy Goryachev
On Tue, 25 Jun 2024 15:14:14 GMT, Markus Mack 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,

Re: RFR: 8314754: Minor ticks are not getting updated both the axes in LineChart

2024-06-25 Thread Andy Goryachev
On Tue, 25 Jun 2024 15:14:14 GMT, Markus Mack 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,