RFR: JDK-8327727: Changing the row factory of a TableView does not recreate the rows

Marius Hanl mhanl at openjdk.org
Sat Mar 9 19:56:18 UTC 2024


The fix is very straightfoward. We request a recreation of all cells in the `VirtualFlow` by calling `flow.recreateCells()` when the `tableRowFactory` was changed.

Funny enough, this was done in the `TreeTableViewSkin` again, hence it did not suffer from this bug. Removed as the table row factory is already handled in the `TableViewSkinBase`, now with the correct recreation.

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

Commit messages:
 - JDK-8327727: Changing the row factory of a TableView does not recreate the rows

Changes: https://git.openjdk.org/jfx/pull/1396/files
 Webrev: https://webrevs.openjdk.org/?repo=jfx&pr=1396&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8327727
  Stats: 98 lines in 4 files changed: 93 ins; 4 del; 1 mod
  Patch: https://git.openjdk.org/jfx/pull/1396.diff
  Fetch: git fetch https://git.openjdk.org/jfx.git pull/1396/head:pull/1396

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


More information about the openjfx-dev mailing list