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

ListViewSkin et al: must not cancel edit on scrolling

XMLWordPrintable

      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.

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

              Created:
              Updated:
              Resolved: