On Sat, 22 Jun 2024 10:55:28 GMT, Markus Mack wrote:
>> This PR provides the test case given in the JBS issue, and a simple fix for
>> the index calculation when inserting data after previous data with duplicate
>> categories.
>>
>> Also, I've added a comment to `BarChart`s javadoc, clarifying
On Sat, 22 Jun 2024 10:55:28 GMT, Markus Mack wrote:
>> This PR provides the test case given in the JBS issue, and a simple fix for
>> the index calculation when inserting data after previous data with duplicate
>> categories.
>>
>> Also, I've added a comment to `BarChart`s javadoc, clarifying
On Sat, 22 Jun 2024 10:55:28 GMT, Markus Mack wrote:
>> This PR provides the test case given in the JBS issue, and a simple fix for
>> the index calculation when inserting data after previous data with duplicate
>> categories.
>>
>> Also, I've added a comment to `BarChart`s javadoc, clarifying
On Sat, 22 Jun 2024 10:58:53 GMT, Markus Mack wrote:
> I can reproduce this, looks unrelated to the issue this PR is trying to fix.
Created https://bugs.openjdk.org/browse/JDK-8334873
-
PR Comment: https://git.openjdk.org/jfx/pull/1476#issuecomment-2186856495
On Sat, 22 Jun 2024 10:55:28 GMT, Markus Mack wrote:
>> This PR provides the test case given in the JBS issue, and a simple fix for
>> the index calculation when inserting data after previous data with duplicate
>> categories.
>>
>> Also, I've added a comment to `BarChart`s javadoc, clarifying
On Sat, 22 Jun 2024 10:52:42 GMT, Markus Mack wrote:
>> modules/javafx.controls/src/main/java/javafx/scene/chart/BarChart.java line
>> 227:
>>
>>> 225: var uniqueCategories = new HashSet();
>>> 226: for (var entry : seriesCategoryMap.entrySet()) {
>>> 227:
On Sat, 22 Jun 2024 10:55:28 GMT, Markus Mack wrote:
>> This PR provides the test case given in the JBS issue, and a simple fix for
>> the index calculation when inserting data after previous data with duplicate
>> categories.
>>
>> Also, I've added a comment to `BarChart`s javadoc, clarifying
On Sat, 22 Jun 2024 10:55:28 GMT, Markus Mack wrote:
>> This PR provides the test case given in the JBS issue, and a simple fix for
>> the index calculation when inserting data after previous data with duplicate
>> categories.
>>
>> Also, I've added a comment to `BarChart`s javadoc, clarifying
> This PR provides the test case given in the JBS issue, and a simple fix for
> the index calculation when inserting data after previous data with duplicate
> categories.
>
> Also, I've added a comment to `BarChart`s javadoc, clarifying the behavior
> that was apparently assumed (but broken) pr
On Fri, 21 Jun 2024 22:07:22 GMT, Andy Goryachev wrote:
>> Markus Mack has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> testAddingDataAtIndex & fix
>
> modules/javafx.controls/src/main/java/javafx/scene/chart/BarChart.java line
> 227:
>
On Fri, 21 Jun 2024 22:10:42 GMT, Markus Mack wrote:
>> This PR provides the test case given in the JBS issue, and a simple fix for
>> the index calculation when inserting data after previous data with duplicate
>> categories.
>>
>> Also, I've added a comment to `BarChart`s javadoc, clarifying
On Fri, 21 Jun 2024 22:07:22 GMT, Markus Mack wrote:
>> This PR provides the test case given in the JBS issue, and a simple fix for
>> the index calculation when inserting data after previous data with duplicate
>> categories.
>>
>> Also, I've added a comment to `BarChart`s javadoc, clarifying
On Fri, 21 Jun 2024 15:22:41 GMT, Markus Mack wrote:
>> This PR provides the test case given in the JBS issue, and a simple fix for
>> the index calculation when inserting data after previous data with duplicate
>> categories.
>>
>> Also, I've added a comment to `BarChart`s javadoc, clarifying
> This PR provides the test case given in the JBS issue, and a simple fix for
> the index calculation when inserting data after previous data with duplicate
> categories.
>
> Also, I've added a comment to `BarChart`s javadoc, clarifying the behavior
> that was apparently assumed (but broken) pr
On Fri, 21 Jun 2024 15:22:41 GMT, Markus Mack wrote:
>> This PR provides the test case given in the JBS issue, and a simple fix for
>> the index calculation when inserting data after previous data with duplicate
>> categories.
>>
>> Also, I've added a comment to `BarChart`s javadoc, clarifying
On Fri, 21 Jun 2024 15:22:41 GMT, Markus Mack wrote:
>> This PR provides the test case given in the JBS issue, and a simple fix for
>> the index calculation when inserting data after previous data with duplicate
>> categories.
>>
>> Also, I've added a comment to `BarChart`s javadoc, clarifying
On Thu, 20 Jun 2024 17:44:41 GMT, Markus Mack wrote:
>> This PR provides the test case given in the JBS issue, and a simple fix for
>> the index calculation when inserting data after previous data with duplicate
>> categories.
>>
>> Also, I've added a comment to `BarChart`s javadoc, clarifying
> This PR provides the test case given in the JBS issue, and a simple fix for
> the index calculation when inserting data after previous data with duplicate
> categories.
>
> Also, I've added a comment to `BarChart`s javadoc, clarifying the behavior
> that was apparently assumed (but broken) pr
On Thu, 20 Jun 2024 19:11:48 GMT, Markus Mack wrote:
> adding empty series
ah, right!
> the tick marks on the axis are in incorrect
this might be [JDK-8314754](https://bugs.openjdk.org/browse/JDK-8314754)
-
PR Comment: https://git.openjdk.org/jfx/pull/1476#issuecomment-21813763
On Thu, 20 Jun 2024 17:44:41 GMT, Markus Mack wrote:
>> This PR provides the test case given in the JBS issue, and a simple fix for
>> the index calculation when inserting data after previous data with duplicate
>> categories.
>>
>> Also, I've added a comment to `BarChart`s javadoc, clarifying
On Thu, 20 Jun 2024 17:44:41 GMT, Markus Mack wrote:
>> This PR provides the test case given in the JBS issue, and a simple fix for
>> the index calculation when inserting data after previous data with duplicate
>> categories.
>>
>> Also, I've added a comment to `BarChart`s javadoc, clarifying
On Thu, 20 Jun 2024 17:44:41 GMT, Markus Mack wrote:
>> This PR provides the test case given in the JBS issue, and a simple fix for
>> the index calculation when inserting data after previous data with duplicate
>> categories.
>>
>> Also, I've added a comment to `BarChart`s javadoc, clarifying
On Thu, 20 Jun 2024 17:44:41 GMT, Markus Mack wrote:
>> This PR provides the test case given in the JBS issue, and a simple fix for
>> the index calculation when inserting data after previous data with duplicate
>> categories.
>>
>> Also, I've added a comment to `BarChart`s javadoc, clarifying
On Thu, 20 Jun 2024 17:59:13 GMT, Andy Goryachev wrote:
>> You'll need to add an additional point after adding the one with duplicate
>> categories to see the exception in the console. Using the "Add Point" button
>> in the monkey tester after using the context menu entry should reproduce the
On Thu, 20 Jun 2024 17:44:41 GMT, Markus Mack wrote:
>> This PR provides the test case given in the JBS issue, and a simple fix for
>> the index calculation when inserting data after previous data with duplicate
>> categories.
>>
>> Also, I've added a comment to `BarChart`s javadoc, clarifying
On Thu, 20 Jun 2024 17:50:20 GMT, Markus Mack wrote:
>> I've added a popup menu on the BarChart page in the Monkey Tester to test
>> the case ...
>> https://github.com/andy-goryachev-oracle/MonkeyTest
>
> You'll need to add an additional point after adding the one with duplicate
> categories to
On Thu, 20 Jun 2024 16:52:49 GMT, Andy Goryachev wrote:
>> However, I don't see the issue if I just add a duplicate data point after
>> the chart is rendered (the code jumps to line 218 (edit: 216, sorry) in the
>> master branch).
>>
>> This makes me think that perhaps the fix should include f
> This PR provides the test case given in the JBS issue, and a simple fix for
> the index calculation when inserting data after previous data with duplicate
> categories.
>
> Also, I've added a comment to `BarChart`s javadoc, clarifying the behavior
> that was apparently assumed (but broken) pr
On Thu, 20 Jun 2024 16:34:23 GMT, Andy Goryachev wrote:
>> This PR provides the test case given in the JBS issue, and a simple fix for
>> the index calculation when inserting data after previous data with duplicate
>> categories.
>>
>> Also, I've added a comment to `BarChart`s javadoc, clarify
On Wed, 19 Jun 2024 10:21:52 GMT, Markus Mack wrote:
> This PR provides the test case given in the JBS issue, and a simple fix for
> the index calculation when inserting data after previous data with duplicate
> categories.
>
> Also, I've added a comment to `BarChart`s javadoc, clarifying the
On Thu, 20 Jun 2024 16:50:23 GMT, Andy Goryachev wrote:
>> modules/javafx.controls/src/main/java/javafx/scene/chart/BarChart.java line
>> 215:
>>
>>> 213: if (!categoryAxis.getCategories().contains(category)) {
>>> 214: // find category index in case data contains duplicate
On Wed, 19 Jun 2024 10:21:52 GMT, Markus Mack wrote:
> This PR provides the test case given in the JBS issue, and a simple fix for
> the index calculation when inserting data after previous data with duplicate
> categories.
>
> Also, I've added a comment to `BarChart`s javadoc, clarifying the
On Wed, 19 Jun 2024 10:21:52 GMT, Markus Mack wrote:
> This PR provides the test case given in the JBS issue, and a simple fix for
> the index calculation when inserting data after previous data with duplicate
> categories.
>
> Also, I've added a comment to `BarChart`s javadoc, clarifying the
This PR provides the test case given in the JBS issue, and a simple fix for the
index calculation when inserting data after previous data with duplicate
categories.
Also, I've added a comment to `BarChart`s javadoc, clarifying the behavior that
was apparently assumed (but broken) previously.
T
34 matches
Mail list logo
