I don't think line 49 is being fired, so I don't t...
# tornadofxa
amanda.hinchman-dominguez
03/03/2019, 11:02 PMI don't think line 49 is being fired, so I don't think I'm calling this correctly
o
Olav Hermansen
03/03/2019, 11:05 PM48 is empty line?
a
amanda.hinchman-dominguez
03/03/2019, 11:06 PMwhoops 49
m
Mark Stefanovic
03/04/2019, 9:21 PMon line 54, shouldn't the scope be passed in to the fire too? line 49 looks fine, although you could use action { } instead and save a few characters since it just calls setOnAction anyways
a
amanda.hinchman-dominguez
03/04/2019, 9:22 PMI'm not 100% sure, when I was looking at the eventbus stuff, it said in some cases it is not necessary: https://github.com/edvin/tornadofx-guide/blob/master/part2/EventBus.md
amanda.hinchman-dominguez
03/04/2019, 9:23 PMalthough you code be right
amanda.hinchman-dominguez
03/04/2019, 9:23 PMI think I'll have to look at it again
m
Mark Stefanovic
03/04/2019, 9:28 PMI made a project where I took the event bus to the Nth degree just to experiment with it, it wasn't as disasterous as I expected: https://github.com/MarkStefanovic/HealthNut
a
amanda.hinchman-dominguez
03/04/2019, 9:29 PMnice! I'll take a look. Thanks!
amanda.hinchman-dominguez
03/04/2019, 9:47 PMThis project looks really cool, maybe I'll use it myself to keep myself accountable against all the snacks at work 😄
amanda.hinchman-dominguez
03/04/2019, 9:47 PMCan I share this on twitter?
m
Mark Stefanovic
03/04/2019, 9:47 PMsure 😃
a
amanda.hinchman-dominguez
03/04/2019, 10:20 PMI wanted to give this a spin, does this project run on Android SDK?
m
Mark Stefanovic
03/05/2019, 12:58 AMthat's a good question, I've never tried, probably not
a
amanda.hinchman-dominguez
03/05/2019, 2:05 AMOh okay! Weird idea was asking for it. Not sure why. I'll try again in a bit