Cool, I managed to get it down to this:

@User uploaded a file: A data grid that only creates the graphic node once per item

You can even combine this with further augmenting this on each updateItem run:

@User uploaded a file: Cached graphic and dynamic cellFormat

The real power over the conventional approach is that due to the nature of Cells in JavaFX, you normally would have a performance nightmare without manually tracking and caching the ImageView instances.

The complete thing is a little less than 500 lines of code, so it's quite large, but definitely worth it. One of the reasons it's so huge is because of all the ceremony of classes to do this the JavaFX way (TM). It would be 100 lines less if I used a more TornadoFX-like approach but I think it's worth it. As the features grows we will benefit from this structure.

Can't wait to read the code and learn from it.

code:

@User uploaded a file: Untitled

adding cellWidth and cellHeight solves this:

@User uploaded a file:

not using the cache: the image is not centered anymore

@User uploaded a file: and commented: Datagrid crashes

@User uploaded a file: Stacktrace of crash

@User uploaded a file: code that makes datagrid crash

So you noticed I committed it @ron 🙂 Haha, it is far from done, but actually it is good that you tested it. I hadn't catched that bug myself 🙂 I will probably not have much time to continue work on it today (my daughter's birthday party no 1). Will let you know when I have ironed out that particular bug, but there are many more as it is not complete yet.

@ron Did you change the content of the list by assigning a new list to

datagrid.items

or did you call

datagrid.items.setAll

when you change between albums?

(The bug has to to with the missing listeners for everything that can change in a datagrid during it's life time. I'm aware of that several are missing, working on it :)

I just found out that every time you resize the grid, another set of duplicate select listeners are added.

I did items = otherlist

@ron I did some adjustments. Probably did not solve your issue, but if you could try it, it might throw an error that gets me closer to fixing the issue with some cells not rerendering correctly.

ok

what exactly do you want to have tested?

Just do what you did. Change the items, check that selection works, check that resizing works etc.

I suspect you will have redraw issues after changing the list, that's what I'm looking at now.

Hm.. maybe I should add built in progress info so you can run long running tasks like loading large images and have nice feedback while you wait..

hmmz dont know what you did, but it is worse now

Great 🙂 haha

What happens?

@User uploaded a file: