RFR: 8283675: Line not removed from LineChart when series cleared
Ajit Ghaisas
aghaisas at openjdk.org
Fri Sep 1 10:41:43 UTC 2023
On Fri, 18 Aug 2023 07:25:28 GMT, Karthik P K <kpk at openjdk.org> wrote:
> The issue is present in AreaChart along with the LineChart. Issue is fixed in both the charts as part of this PR.
> The line elements in case of Line chart and both line element and fill element in the case of Area charts were not cleared in `makePaths()` method in AreaChart.java. Hence the line and fill area were not getting cleared when series was cleared.
>
> Made changes in code to clear line element and fill element as required in the `makePaths()` method.
>
> Added tests to validate the changes in both LineChart and AreaChart.
The fix looks good to me!
-------------
Marked as reviewed by aghaisas (Reviewer).
PR Review: https://git.openjdk.org/jfx/pull/1214#pullrequestreview-1606704659
More information about the openjfx-dev
mailing list