RFR: 8265206: Tree-/TableCell: editing state not updated on cell re-use [v2]

Jeanette Winzenburg fastegal at openjdk.java.net
Fri Apr 23 15:31:52 UTC 2021


> Issue is missing update of Tree-/TableCell's editiable state when changing its index from editingIndex to -1. 
> 
> Seems to be a left-over from fixing cell's editing update - done in [JDK-8150525](https://bugs.openjdk.java.net/browse/JDK-8150525) - on index change for the special case of new index -1.
> 
> Fixed by cleaning out editing state in that corner case also, added tests that were failing before and passing after the fix. Note that there are also tests that passed before: the previous fix didn't add any tests, so added them here.

Jeanette Winzenburg has updated the pull request incrementally with one additional commit since the last revision:

  changed doCancelEdit to reliably reset the flag updateEditingIndex

-------------

Changes:
  - all: https://git.openjdk.java.net/jfx/pull/473/files
  - new: https://git.openjdk.java.net/jfx/pull/473/files/0d2fa43a..e861a72b

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jfx&pr=473&range=01
 - incr: https://webrevs.openjdk.java.net/?repo=jfx&pr=473&range=00-01

  Stats: 164 lines in 4 files changed: 158 ins; 0 del; 6 mod
  Patch: https://git.openjdk.java.net/jfx/pull/473.diff
  Fetch: git fetch https://git.openjdk.java.net/jfx pull/473/head:pull/473

PR: https://git.openjdk.java.net/jfx/pull/473


More information about the openjfx-dev mailing list