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

    ron

    01/07/2018, 7:28 PM
    @ron uploaded a file: Untitled
    -.kt
  • r

    ron

    01/07/2018, 7:28 PM
    when calling the disableNavigation
  • e

    edvin

    01/07/2018, 7:30 PM
    @ron Show me the code 🙂
  • r

    ron

    01/07/2018, 7:30 PM
    @ron uploaded a file: Untitled
    -.kt
  • r

    ron

    01/07/2018, 7:30 PM
    if I dont call the disableNavigation it works fine
  • e

    edvin

    01/07/2018, 7:33 PM
    This is too early, you must do it in the init of your custom workspace, if you have one, or atleast at a later stage than init of the app class.
  • r

    ron

    01/07/2018, 7:36 PM
    I am just going to hide the buttons
  • e

    edvin

    01/07/2018, 7:38 PM
    That works too 🙂
  • r

    ron

    01/07/2018, 7:38 PM
    @edvin when you use the WorkspaceApp where do you override the onSave of the workspace?
  • e

    edvin

    01/07/2018, 7:40 PM
    @ron You'd have to set 
    FX.defaultWorkspace
    to point to your custom instance 🙂
  • e

    edvin

    01/07/2018, 7:42 PM
    @ron I just committed a fix for the 
    workspace.disableNavigation()
    issue - it was actually a bug 🙂
  • r

    ron

    01/07/2018, 7:55 PM
    good, I went back to just BpWorkspace : Workspace()
  • r

    ron

    01/07/2018, 7:55 PM
    feels more natural to do the override save and such there
  • a

    amanda.hinchman-dominguez

    01/07/2018, 10:02 PM
    does evt.consume really stops propagation or am I not understanding as well as I should?
  • a

    amanda.hinchman-dominguez

    01/07/2018, 10:03 PM
    @amanda.hinchman-dominguez uploaded a file: Untitled
    -.js
  • a

    amanda.hinchman-dominguez

    01/07/2018, 10:07 PM
    whoops, I think I got it
  • r

    ron

    01/07/2018, 10:07 PM
    @edvin when you feel like it have a look at https://github.com/ronsmits/bpgraph it has add, delete and update (update by double clicking on a row)
  • a

    amanda.hinchman-dominguez

    01/07/2018, 10:08 PM
    It looks like you can't place an event.consume AFTER an alert - in the debugger, I noticed event.consume wasn't being hit being placed before the alert
  • r

    ron

    01/07/2018, 10:15 PM
    so it does work when you put it in front of the alert?
  • e

    edvin

    01/07/2018, 10:18 PM
    @amanda.hinchman-dominguez Alert does 
    showAndWait()
    so you should definitely not do that in the same pulse as an event handler. Wrap it in a 
    runLater {}
    block.
  • a

    amanda.hinchman-dominguez

    01/07/2018, 10:18 PM
    what is a 
    showAndWait()
    ?
  • a

    amanda.hinchman-dominguez

    01/07/2018, 10:19 PM
    I mean, what does it wait for?
  • e

    edvin

    01/07/2018, 10:19 PM
    It asks the UI thread to wait with further processing.
  • c

    carlw

    01/07/2018, 11:24 PM
    What’s wrong with the show and wait? @edvin you can put a confirmation dialog in an action, right?
  • a

    amanda.hinchman-dominguez

    01/08/2018, 12:09 AM
    @carlw To answer the later question, you can - I did (where a success drops the tile in the grid, a failure creates an error dialog)...right now, the failure creates the dialog, but it looks like the the helper function gets called again...
  • a

    amanda.hinchman-dominguez

    01/08/2018, 12:19 AM
    @amanda.hinchman-dominguez uploaded a file: Untitled
    -.js
  • c

    carlw

    01/08/2018, 12:20 AM
    what do you need the evt.consume for?
  • a

    amanda.hinchman-dominguez

    01/08/2018, 12:49 AM
    to stop the process from continuing in a string of functions, but it was not effective until I place evt.consume before the alert
  • r

    ron

    01/08/2018, 10:22 AM
    @edvin I think what I checked in yesterday is how you thought it should be done
  • e

    edvin

    01/08/2018, 10:23 AM
    @ron Will check after lunch, heading out now!
1...162516261627...2007Latest