do not touch the ring of wagamoto

Le lun. 13 mars 2023 à 08:23, Toshio Nakamura <tnakam...@openjdk.org> a
écrit :

> On Thu, 26 Jan 2023 09:15:47 GMT, Toshio Nakamura <tnakam...@openjdk.org>
> 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
>

Reply via email to