Was just about to paste that from some code I had written..

@User uploaded a file: Add onCommit handler to all columns

This of course calls for a TornadoFX convenience function.

I feel I prefer this:

It will also listen for changes, so if you add columns after this call, they will still participate in the commit callback.

so that is on TableView?

Yes. But it seems to fire to early. When my event handler hits, the domain object still has the old value. It gets the new value right after. Investigating now.

I confirmed that the event is indeed fired before the data is updated in TableCell.. 😞

When I did it - I had to use Platform.runLater otherwise I had the exact same issue

That's what I did as well 🙂

Hehe

It's working correctly now.

@gtnarg Thank you for this, that was a very nice addition to the TableView support.

glad I could be assistance for once 🙂

I'm updating orderlines in our new system running on postgresql. Any update will do a two phase commit against postgresql and the old system running firebirdsql. The schemas are completely different, so data is converted into something that will convey the same meaning in the old system. Crazy. Can't wait to be done with the old system and throw away this two pase commit thing. It's working, but it feels dangerous 🙂

@edvin just talked with our competence manager and I will submit a paper for devoxx belgium

group9 ( company I work for) will sponsor and assist it

Wow, that's fantastic!

the fact that we weill do a call for paper does not automatically mean it will be accepted of course

I know

oh and probably tomorrow I will have a youtube link for you from the miniconf

Nice! Can't wait @ron 🙂

I'm trying to create a type safe way to express select bindings. That would be very neat.

@User uploaded a file: Untitled

This is pretty convenient compared to a select binding.

@edvin is there a reason that there isn't a single no-arg menucheckitem or radio? I have a few of these and have to add in null unlike the regular menuitem (which has 2 constructors) If not, should I write up an issue? checkmenuitem("Show Profile Browser", null, null)

@carlw I've committed a fix 🙂

thx

👍 It was actually just a default value that was missing (keyCombination could be null but was required non the less)