TableView#useCheckBox() not binding

[andrzejtokarski]

GitHub does not let me to reopen the issue so here is a new one.

To replicate:

import tornadofx.*

class CheckBoxTest : App(CheckBoxTestWindow::class)

class CheckBoxTestWindow: View() {

    class SomeViewModel(id: String, ignore: Boolean) {
        var id by property(id)
        var ignore by property(ignore)
    }

    val elements = (0..100).map { SomeViewModel(it.toString(), ignore = false) }.toMutableList().observable()

    override val root = tableview(elements) {
        column("Id", SomeViewModel::id)
        column("Ignore", SomeViewModel::ignore).useCheckbox(editable = true)
    }
}

Check some of the boxes then scroll away and back. Those boxes are back to false.

[andrzejtokarski]

My fault. The view model should include:
fun ignoreProperty() = getProperty(SomeViewModel::ignore)

and column definition:
column("Ignore", SomeViewModel::ignoreProperty).useCheckbox(editable = true)

as mentioned in the documentation:
https://edvin.gitbooks.io/tornadofx-guide/content/5.%20Data%20Controls.html

[edvin]

Actually, there is no reason why we shouldn't support your initial code as well. I've improved the targeting for mutable columns, so the code example above should now work out of the box with 1.6.2-SNAPSHOT. Thanks for reporting this :)

[andrzejtokarski]

This is why this project is amazing. If only Oracle cared half this much...

[edvin]

Let's hope it picks up steam after JDK 9 is out. I think a lot of work went into adapting for Jigsaw, even if it produced minimal features or improvements.