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

    alexp11223

    05/27/2017, 3:36 PM
    @User uploaded a file: Untitled
    -.kt
  • e

    edvin

    05/27/2017, 3:57 PM
  • a

    alexp11223

    05/27/2017, 4:27 PM
    @edvin yeah, it seems to work.
  • a

    alexp11223

    05/27/2017, 4:27 PM
    But now another problem: looks like in code above 
    status
    changes before 
    ui
    . So assert is executed when 
    stuff
    is not set yet
  • a

    alexp11223

    05/27/2017, 4:28 PM
    oh, and looks like that rule somehow swallows exceptions inside 
    runAsync
    ...
  • a

    alexp11223

    05/27/2017, 5:10 PM
    Actually not before 
    ui
    , but before 
    status
    was set in runAsync, or something like that. On first check of 
    if (condition(value)) {
    both old and new values are true (from manual call, 
    changeListener.changed(this, value, value)
    )
  • a

    alexp11223

    05/27/2017, 5:46 PM
    hm, found another test runner, but looks like it swallows async exceptions too https://github.com/sialcasa/jfx-testrunner
  • a

    alexp11223

    05/27/2017, 5:46 PM
    btw looks like it also has something similar to await (
    startAndWaitForValue
    )
  • a

    alexp11223

    05/27/2017, 5:52 PM
    ah, no, that's just me shooting myself into foot =\
  • a

    alexp11223

    05/27/2017, 5:52 PM
    I forgot about 
    fail
    with commented //fire
  • a

    alexp11223

    05/27/2017, 5:55 PM
    actually no, it is in the output but still not propagated. hm, not propagated even if I rethrow it in 
    subscribe<ErrorEvent> {  throw it.error }
  • s

    Sola

    05/27/2017, 10:18 PM
    @edvin have you tested the 
    centerOnScreen
    parameter in 
    replaceWith
    function? cause it doesn't work for me 😂
  • e

    edvin

    05/27/2017, 10:20 PM
    @Sola Did you add a transition when you tried it?
  • s

    Sola

    05/27/2017, 10:21 PM
    I tried both with a transition and without
  • e

    edvin

    05/27/2017, 10:22 PM
    @Sola I think I see the issue, hang on.
  • s

    Sola

    05/27/2017, 10:23 PM
    I saw the code in 
    replaceWith
    , the transition seems doesn't matter about invoking 
    onTransit
    lambda
  • e

    edvin

    05/27/2017, 10:24 PM
    @Sola It happens to early. I've changed it now, can you pull and try?
  • s

    Sola

    05/27/2017, 10:25 PM
    I'm using snapshot, not building from git
  • e

    edvin

    05/27/2017, 10:27 PM
    Aha, hang on.
  • e

    edvin

    05/27/2017, 10:30 PM
    @Sola I've verified that it works locally and I'm building a new snapshot for you now, hold on.
  • e

    edvin

    05/27/2017, 10:32 PM
    It should be available now 🙂
  • s

    Sola

    05/27/2017, 10:53 PM
    I just figured out how to force refresh snapshot, hold on
  • s

    Sola

    05/27/2017, 10:55 PM
    Great! It works fine now.
  • s

    Sola

    05/27/2017, 10:55 PM
    Thank you.
  • t

    thomasnield

    05/27/2017, 11:09 PM
    @edvin @Sola remember you can use JitPack too
  • r

    redrield

    05/28/2017, 2:28 AM
    How can I make sure that any long running async tasks are closed when the UI is destroyed
  • r

    redrield

    05/28/2017, 2:29 AM
    I have a scenario where I have a little form in the beginning, then I feed the input from that into a function that runs until the program ends
  • r

    redrield

    05/28/2017, 2:29 AM
    And I can't have that on the UI thread for obvious reasons
  • r

    redrield

    05/28/2017, 2:29 AM
    But when I put it in runAsync, when I closed the UI, it continued running and I had to force quit the app in IJ
  • c

    carlw

    05/28/2017, 2:48 AM
    @redrield Does the code in runAsync ever end? Are you expecting the closed Stage(s) to also kill the code in your task?
1...122112221223...2007Latest