@gtnarg You could bind to a custom property that reads/writes to the textfield. Implement SimpleStringProperty and override get and set maybe?

@edvin the amazing this was how easy it was to create it... very nice... i need to work on something a little more substantial, but so far tornadofx is very nice... thanks again

@jchildress Yeah, Kotlin in combination with the base we have created in TornadoFX has proven to be gold when we create new enhancements 🙂 If you see any boiler plate in your TornadoFX apps be sure to scream so we can fix it! hehe

will do...

@User uploaded a file: Binding issue solved and commented: @edvin - thanks - that worked 🙂

@gtnarg Great! By the way, we have an

onChange

shortcut instead of the addlistener thing, so you can write that last line

textArea.textProperty().onChange { textProperty.setValueFromEditor(it) }

.

@edvin Looking at removing the MainView reference in the TODO app...I pulled the latest and am still getting the completed() property firing more than once

@User uploaded a file: Untitled and commented: Called more than once from a check box @edvin

How does the whole class look now Carl?

@User uploaded a file: Untitled

@carlw This still has the issue that the binding from the last item this fragment was used for will still be connected. Either it has to be disconnected (I think I put up an example for that) or we have to reintroduce the ViewModel. By the way, the

contentLabel.toggleClass

line should be outside the

itemProperty.onChange

- it doesn't need to be reapplied when the item changes. I think the TornadoFX ViewModel is not very well suited for mediating between Views in it's current implementation actually. Right now a better word for it would probably be

EditModel

, since it doesn't observe changes to the properties it wraps, and that it needs a commit to happen for changes to be pushed back to the underlying object. This use case would be easier solved with a more direct view model. I will create an example and show you what I mean tomorrow. Wow, this little app has made me realize a lot of shortcomings for the ViewModel. It might just need an

immediate

mode though. Will be looking at that tomorrow as well.

ok. i had a contains() check around the stylesheet toggleClass() setting in my original version. i think this version came from the patch

@User uploaded a file: Untitled and commented: Working to replace mainView.updateItemsLeftLabel()

OK. There are still some things I don't fully understand with that's happening here, so I'll let you know when I have them sorted. Sorry that this is taking so long, I've had too many things going on this week. Soon though 🙂

np. i was just bouncing around some of the commits. it looks like the patch broke some things when i use the app for a while

if i add / remove / complete a bunch of items, that count at the very least seems to fall apart

commit c2ae3ee2 in my branch seems to be the last working version

@carlw OK, thanks - will keep you posted.

ok...here's the working fragment and controller in that commit. this is using the readCache() thing from a few weeks ago where i return a specific fragment based on the unique id of the todoitem

@User uploaded a file: Working Fragment

@User uploaded a file: Working Controller

@edvin Do you think it could be caused by the Binding.kt bindBidirectional()? Say there are a bunch of TodoListFragments hanging around all bound to the model (item.completedProperty). If you press the one visible checkbox, the model will update and call the handlers for the discarded checkboxes.

Guys! Hi! I am trying to add tornadoFX to a Java project So the main class is in java The problem is I can't start the interface for some reason

public class Main extends App {
    public Main() {
        super(JvmClassMappingKt.getKotlinClass(MainView.class) );

    }

    public static void main(String [] args)
    {

    }
}

What do I do?

@carlw The problem is not "a bunch of fragments", because there is just one fragment per cell, but in this configuration we reuse the same fragment for different items, so when the item change we must also make sure to unbind from the old item. I gave an example of this some days ago. This is the correct way to handle this, but I want it to be smoother 🙂

@cypher121 So you want the app class to be a java class even if you have Kotlin on the classpath?

Yep

Need to

@cypher121 Can I ask why? I think it's possible to attack this differently.

I just tried this btw and it works:

@User uploaded a file: Untitled