Channels
#tornadofx
- e
edvin
09/03/2016, 5:38 PM@ron But the items in the list change? - e
edvin
09/03/2016, 5:38 PM@ron Or do you change the whole list? In that case you would need to add a change listener to the itemsProperty instead. - r
ron
09/03/2016, 5:38 PMyes I saw it wrong the listener goes off - r
ron
09/03/2016, 5:40 PMok so I can detect when it goes off, now lets see if I can call the populate on it - e
edvin
09/03/2016, 5:56 PM@thomasnield I just figured we can add a column(title: String), so you can use it with `apply`: - t
thomasnield
09/03/2016, 5:56 PMNice, because there would be no second lambda argument? - e
edvin
09/03/2016, 5:56 PMThat's right. - e
edvin
09/03/2016, 5:57 PMWould that be valuable? I find myself using
with columns already, so this would make it a lot cleaner actually.apply - t
thomasnield
09/03/2016, 5:57 PMLet me step back and ask this... - t
thomasnield
09/03/2016, 5:58 PMWhy are we just not deprecating this existing method? - t
thomasnield
09/03/2016, 5:58 PMI don't think anybody really likes it anyway - t
thomasnield
09/03/2016, 5:58 PMAt least I don't - t
thomasnield
09/03/2016, 5:58 PMThat's what we were trying to clean up anyway - t
thomasnield
09/03/2016, 5:58 PMBut then yeah, we would have to make a breaking release - e
edvin
09/03/2016, 5:58 PMYou're right... on both counts. Let me think about that. - t
thomasnield
09/03/2016, 5:59 PMI don't think we can deprecate, we'll just have to rid it. - t
thomasnield
09/03/2016, 6:00 PMGetting rid of that could actually change everything - t
thomasnield
09/03/2016, 6:00 PMfor the better possibly - e
edvin
09/03/2016, 6:00 PMThis one is convenient when you just add the op, but it becomes very ugly when you want to configure more properties on the column, right? - e
edvin
09/03/2016, 6:00 PMThis is not so bad:column("Name") { it.value.name } - e
edvin
09/03/2016, 6:01 PMBut it becomes a mess when you do:column("Name") { it.value.name }.apply { .. more stuff here } - e
edvin
09/03/2016, 6:01 PMOr evencolumn("Name", { it.value.name}).apply { ... } - e
edvin
09/03/2016, 6:02 PMRemoving this and setting the value explicitly would indeed make it cleaner. - e
edvin
09/03/2016, 6:02 PMI have used this a lot in my code though. But I hate looking at that code. - t
thomasnield
09/03/2016, 6:02 PMSot his would probably be for the better removing it. - t
thomasnield
09/03/2016, 6:02 PMMaybe we can deprecate in next release - t
thomasnield
09/03/2016, 6:03 PMand remove it and implement new functions in following release - e
edvin
09/03/2016, 6:03 PMSo then you want to add the one that operates on the column instead?