RFR: 8265210: TreeCell: cell editing state not updated on cell re-use
Ajit Ghaisas
aghaisas at openjdk.java.net
Mon May 24 12:17:01 UTC 2021
On Thu, 13 May 2021 12:22:47 GMT, Jeanette Winzenburg <fastegal at openjdk.org> wrote:
> fix is analogous to similar issues for Tree/TableCell (combined [JDK-8150525](https://bugs.openjdk.java.net/browse/JDK-8150525) and [JDK-8265206](https://bugs.openjdk.java.net/browse/JDK-8265206))
>
> added tests that failed before and passed after the fix
>
> note: this PR also adds two asserts (which I forgot in the previous [PR 473](https://github.com/openjdk/jfx/pull/473);) in Tree/TableEditingTest which are unrelated to this fix.
The fix looks good to me.
I have made one minor test cleanup observation. Once fixed, it it ready to be pushed.
modules/javafx.controls/src/test/java/test/javafx/scene/control/TreeCellEditingTest.java line 45:
> 43: import javafx.scene.control.TreeCell;
> 44: import javafx.scene.control.TreeItem;
> 45: //import javafx.scene.control.TableColumn;
Minor - 2 commented imports can be removed.
-------------
PR: https://git.openjdk.java.net/jfx/pull/501
More information about the openjfx-dev
mailing list