RFR: 8209017: CheckBoxTreeCell: graphic on TreeItem not always showing [v3]

Nir Lisker nlisker at openjdk.org
Tue Jan 3 23:56:12 UTC 2023


> A simple fix for the graphics of a `CheckBoxTreeCell` not showing after expand/collapse events. The graphics node needs to be released from the previous checkbox so that it doesn't serve as the graphics of more than one checkbox at once. If it does, only one checkbox (its skin) can actually use it as a scenegraph child, and it's not always the correct one.
> 
> Added also a manual test app as I'm unsure about how to do an automatic test for this. The test app will be useful for later fixes, so I suggest adding it anyway. To test the fix, simply expand and collapse the root note repeatedly while it has a child. Before the fix the graphic will disappear after a few tries, after the fix it won't. You can test other hierarchies as well by adding children to the tree.

Nir Lisker has updated the pull request incrementally with one additional commit since the last revision:

  Removed thrown exception

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

Changes:
  - all: https://git.openjdk.org/jfx/pull/970/files
  - new: https://git.openjdk.org/jfx/pull/970/files/4d89b0da..abb71bd4

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jfx&pr=970&range=02
 - incr: https://webrevs.openjdk.org/?repo=jfx&pr=970&range=01-02

  Stats: 3 lines in 1 file changed: 0 ins; 2 del; 1 mod
  Patch: https://git.openjdk.org/jfx/pull/970.diff
  Fetch: git fetch https://git.openjdk.org/jfx pull/970/head:pull/970

PR: https://git.openjdk.org/jfx/pull/970


More information about the openjfx-dev mailing list