https://kotlinlang.org logo
Channels
Powered by
#tornadofx
  • e

    edvin

    02/11/2017, 5:18 PM
    @ron The TableView inside the drawer item dictates the size. I'll try to find a way to make it fit the height.
  • g

    gtnarg

    02/11/2017, 5:18 PM
    try removing the 'Tag' type and see if you get the same error.
  • r

    ron

    02/11/2017, 5:19 PM
    column<String>("name", Tag::getTagName)
  • e

    edvin

    02/11/2017, 5:19 PM
    Works?
  • r

    ron

    02/11/2017, 5:19 PM
    @User uploaded a file: Untitled
    -.txt
  • r

    ron

    02/11/2017, 5:20 PM
    nope
  • r

    ron

    02/11/2017, 5:21 PM
    @edvin the table starts as empty
  • r

    ron

    02/11/2017, 5:21 PM
    maybe that info helps
  • e

    edvin

    02/11/2017, 5:21 PM
    @ron Will look at it after dinner.
  • r

    ron

    02/11/2017, 5:22 PM
  • r

    ron

    02/11/2017, 5:25 PM
    @gtnarg the code for this is in https://github.com/ronsmits/KPhoto
  • r

    ron

    02/11/2017, 5:26 PM
    when you start it up it will always load the photos in src/main/resources
  • g

    gtnarg

    02/11/2017, 5:42 PM
    @ron does the initial code using the property name as a String work?
  • r

    ron

    02/11/2017, 5:42 PM
    column<Tag, String>("name", "tagName")
    this you mean, that works fine
  • g

    gtnarg

    02/11/2017, 5:44 PM
    I don't think the method reference is supported for Pojo's although it could probably be added
  • g

    gtnarg

    02/11/2017, 5:47 PM
    actually - I think I spoke too soon - looking at it now...
  • r

    ron

    02/11/2017, 5:50 PM
    btw the 
    wrappingTextProperty
    result is ugly
  • r

    ron

    02/11/2017, 5:51 PM
    @User uploaded a file:

    Pasted image at 2017-02-11, 6:51 PM▾

  • g

    gtnarg

    02/11/2017, 6:07 PM
    @User uploaded a file: Pojo column binding and commented: The following can be added to ItemControls.kt
    Pojo_column_binding.kt
  • g

    gtnarg

    02/11/2017, 6:10 PM
    @edvin - I can create PR for this - or you can just add it. Will also need to create a sister function for TreeTableView...
  • g

    gtnarg

    02/11/2017, 6:10 PM
    @ron - you could test it locally in your class
  • e

    edvin

    02/11/2017, 6:23 PM
    Great work again, Grant!
  • r

    ron

    02/11/2017, 6:23 PM
    I will after dinner
  • e

    edvin

    02/11/2017, 6:25 PM
    @ron I'm looking at a way to optionally make the SqueezeBox folds take up remaining space.
  • e

    edvin

    02/11/2017, 6:25 PM
    What's a better word than 
    useRemainingHeight
    ?
  • r

    ron

    02/11/2017, 6:26 PM
    Use All height?
  • e

    edvin

    02/11/2017, 6:35 PM
    That's it 🙂
  • e

    edvin

    02/11/2017, 6:37 PM
    Maybe fillHeight is better?
  • e

    edvin

    02/11/2017, 6:39 PM
    @ron Committed now. Can you try to set 
    fillHeight = true
    in the squeezebox builder?
  • r

    ron

    02/11/2017, 6:39 PM
    After dinner
1...870871872...2007Latest