Uploaded image for project: 'JDK'
  1. JDK
  2. JDK-8272118

ListViewSkin et al: must not cancel edit on scrolling

    XMLWordPrintable

    Details

      Description

      this is basically reverting JDK-8114594 which cancels the edit on mouse interaction with the scrollBars. That bug is pre-historic (fixed in 2011, version fx2.0, before first commit accessible in the repository), has no description nor any tests.

      As noted in a comment to that issue, it's inconsistent/incomplete, behavior depends on how scrolling is done:

      - mousePressed on scrollBar: edit cancelled
      - by mousewheel/touchPad: does nothing
      - programmatically by control.scrollTo or scrollBar.setValue: does nothing

      My guess is that due to incomplete/missing update of editing state on cell re-use, edits were committed on incorrect items, leading to data corruption. After fixing these re-use issues (see JDK-8264127 and related), the brute-force cancel should no longer be needed.

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              fastegal Jeanette Winzenburg
              Reporter:
              fastegal Jeanette Winzenburg
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: