@thomasnield Yes, we have some key players that are vital to how things work right now, but we invest a lot in systems as well, and those are even more valuable to a large hosting player IMO. Will be exciting to see what happens this time 🙂

Keep us posted! I'm excited for your prospects...

I finished the last functional requirement for the TODO app (filtering by toggle button) and cleaned up some of the code from the suggestions here and from the intellij code review

i'll consolidate the model into a single data source tonight, break the code out into modules, and look at the builder vs fxml

I am bummed now that I have meetings till 9pm today, I would love to remove the fxml and make it with the builders

@ron I won't be working on this for a while, so send me a pull request if you add the builders

@carlw Cool, let me know when you have broken it into separate files, then I'll go over it as well 🙂

@thomasnield I added some info about

ListView.cellCache

to the guide

@ron Call in sick?

I have updated TornadoFX to use Kotlin 1.0.4

@edvin Would it be worth it to add

onChange

overrides for primitive observables (e.g.

ObservableIntegerValue

that uses

op: (Int) -> Unit

instead of

op: (Int?) -> Unit

)?

@Ruckus You would get a KotlinNullPointerException if you used the current

onChange

and the new value is null, right? In that case, yes I think so.

No. currently, the

op

uses

Number?

instead of

Int

, so you just have to ensure it isn't null and cast it to an int, but for

ObservableIntegerValue

, we already know that the return is a non null integer wrapped in a

Number

.

At least, I believe that to be the case. Let me do some quick checks.

Ah, I see. OK.

I'm working on the alternate properties style for the plugin, trying to figure out how I can add a configuration screen for the plugin.

@User uploaded a file:

Nice 🙂

@edvin After updating to Kotlin 1.0.4, the TornadoFX plugin throws an error every few seconds:

org.jetbrains.kotlin.asJava.LightClassUtilsKt.toLightClass(Lorg/jetbrains/kotlin/psi/KtClassOrObject;)Lorg/jetbrains/kotlin/asJava/KtLightClass;

Do you want the whole stack trace?

Hm.. Yes please.. But do you have the Kotlin plugin version 1.0.4 as well?

Yes. I updated it and restarted IntelliJ, which is when I started seeing the errors.

@User uploaded a file: Untitled

I guess they changed something.. hmm.. OK, thanks - will check it out locally. Maybe we should revert in the mean time..

I'll see if it's something with my setup

@Ruckus I reverted to 1.0.3 - can you check if the error goes away? If not, it is probably something else.

Double check that you have plugin version 1.5.6 as well.

I can verify I have plugin v1.5.6

Did you not see the same issue?

(I just invalidated and restarted. I'll pull and check if the error goes away once IntelliJ is done indexing and scanning.)