[Rev 01] RFR: 8193800: TreeTableView selection changes on sorting

Ambarish Rapte arapte at openjdk.java.net
Mon Jun 15 19:25:52 UTC 2020

On Thu, 4 Jun 2020 11:16:50 GMT, Ambarish Rapte <arapte at openjdk.org> wrote:

> In the above case, The selection updates correctly but the received change event on SelectedItems is incorrect. As of
> now it looks like the selection is updated correctly with this fix, but the event generated from sort() method is
> always incorrect. Looking in to fix the change event....

Please take a look at the updated changes. The `GenericAddRemoveChange` event generated from `TreeTableView.sort()`
method does not result in correct selection change events. @kevinrushforth , Thanks for the guidance on this in offline
discussion. The solution is to sort the tree of selected items upfront before generating the `GenericAddRemoveChange`
event. Rest of the TreeItems can be left to be sorted lazily whenever accessed.


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

More information about the openjfx-dev mailing list