RFR: 8295248: JEditorPane HTML form with multi selection broke drawing after reseting [v2]
Toshio Nakamura
tnakamura at openjdk.org
Mon Oct 24 02:06:25 UTC 2022
On Fri, 21 Oct 2022 22:51:49 GMT, Damon Nguyen <dnguyen at openjdk.org> wrote:
>> Toshio Nakamura has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Update testcase
>
> test/jdk/javax/swing/text/html/TestResetSelectForm.java line 93:
>
>> 91: if (getMaxColorDiff(baseColor, c) > LIMIT) {
>> 92: throw new RuntimeException("Unexpected List selection at " +
>> 93: (i+1) + ", " + baseColor + "," + c);
>
> The points mentioned seem valid since I see the same things when I test it myself. OptionListModel's clearSelection is being called without the drawing updating. The fix seems logical and works as intended.
>
> Just to nitpick, adding a space before and after the '+' in '(i+1)' here would match the pattern of all the other additions.
@DamonGuy Thank you for the comments. Fixed.
-------------
PR: https://git.openjdk.org/jdk/pull/10685
More information about the client-libs-dev
mailing list