On Thu, 26 Jan 2023 09:15:47 GMT, Toshio Nakamura <[email protected]> wrote:
>> When JEditorPane shown HTML form with multi-selection, the reset operation >> broke its data. >> >> The sample testcase were attached in JBS. The recreate steps are below: >> 1. The sample shows a multi-selection form with 5 items. >> 2. Select the 4th item. >> 3. Press the reset button. >> Actual: The 4th item is still selected. >> Expect: No item should be selected. Resetting operation may move to the >> initial state. The sample doesn't use 'selected' property in each option tag. >> 4. Just after the step 3, move the focus to outside of the application and >> back. >> Actual: The 2nd item is selected. It means inside data indicates so. >> Expect: No item is selected. >> >> There are two issues. The current method >> `OptionListModel.removeIndexInterval()` removed the indexed item, and >> shifted the rest. Then, selection values were corrupted. >> I think the clear selection method `OptionListModel.clearSelection()` is >> suitable here. >> >> Test: jdk_desktop on macOS (x64, Monterey), Linux (x64, RHEL8), and Windows >> (x64, 2012R2). >> No regression found > > Toshio Nakamura has updated the pull request with a new target base due to a > merge or a rebase. The incremental webrev excludes the unrelated changes > brought in by the merge/rebase. The pull request contains five additional > commits since the last revision: > > - Update Copyright years > - Merge branch 'master' into 8295248 > - Fixed space > - Update testcase > - 8295248: JEditorPane HTML form with multi selection broke drawing after > reseting Ping for keeping this PR active. Could you tell me what this PR is missing? ------------- PR: https://git.openjdk.org/jdk/pull/10685
