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