I’ve actually contemplated putting the view layer into a separate maven module just to prevent reverse dependencies (i.e model to view) - perhaps this kind of this can more easily be accomplished with java 9 ?

@gtnarg That's a very interesting idea actually.

We'll fork for JDK9 after the coroutines stuff is settled.

@gtnarg I'd keep the view and model components together, but definitely draw out any data structures into a commons project to that they can be used in other areas like web services, ejbs or backend logic, webapps, command line apps, etc. Maybe a service contract interface w/o annotations can be factored out

@carlw - that’s generally how I partition it

the difference in model components - say a RESTful client used by JavaFX and one used by command line tool -- has to do with wiring, say how the API URL is configured. This might hamper their reuse to justify a second JAR.

But definitely pay attention to DRY

@gtnarg unfortunately it's not tornadofx, but i'm having great results with MVVM. the one-way strategy to avoid cascading updates is found in other architectures too like Flux / React

@carlw - I agree - but it’s certainly easy to support with TornadoFx

definitely. i'm held back on kotlin w. my existing clients as a sticking point

@carlw Because they want to maintain the code themselves and don't know Kotlin?

I uploaded my TodoMVC version: https://github.com/edvin/todomvc

Any feedback would be welcome. I'm planning a screencast tonight.

@edvin Awesome, I will have a look at it and hope to learn something from the master 😉

@nimakro Great! I will explain my reasoning for everything in the screencast, but even without it I think the intent of this code is very easy to grasp. I will also add comments to this code, but I'd like to do the screencast without it first.

@edvin Is TornadoFX 1.7.1-SNAPSHOT compiling for you because I get Errors in AutoCompleteComboBoxSkin.kt:

Yeah, it's compiling for me. You have the latest commit?

Doing a clean/build from the command line here now.

Make sure you clean first, btw

Yes I am up to date. Mhm

It seems that it can't find the reference to com.sun.javafx.collections.annotations.ReturnsUnmodifiableCollection, Let me check I might have some problems with JavaFx 🙈

Which JDK are you using?

@nimakro That annotation is not used in TornadoFX. It is however used extensively in JavaFX itself. I'm running the latest Oracle JDK.

Maybe you're on OpenJDK without JavaFX?

No I am running java-8-jdk so that shouldn't be the problem.

I get an error in ItemControls.kt line 613 that tableView calls the private refresh() methode of tableView strange.

That's not private in JDK8?

Make sure you have at least 8u60 though, that's when it was introduced.

Off to see Beauty and the Beast 🙂