I'll do that then. Thanks for the help.You're the best @edvin

movie time 🙂

@permalmberg Enjoy! Same here 🙂

Is the Layout Debugger working for anyone with the 1.5.10-SNAPSHOT? I tried it on a few applications and get the following exception: "java.lang.IllegalArgumentException: Children: duplicate children added: parent = BorderPane@70698b22"

It works with 1.5.9

@jchildress: Thank you, I'll correct it ASAP :)

@edvin No problem. Thanks again for an awesome framework. You guys are amazing. I agree with @thomasnield about your writing too... thanks.

@jchildress: Thank you, it really means a lot :)

Something with the paddingLeft and friends are bothering me. They are functions, when they could have been vars. That way you could also read out the property. I'll try to add them as properties and deprecate the functions.

Done. IDEA will now suggest the replacement as a quickfix as well.

@jchildress Fixed the typos now, thanks again 🙂

@edvin No Problem. On Chapter 11 now...

@jchildress Oh, great! I can confirm I see the issue with the Layout Debugger, will try to resolve that tomorrow.

Ok, no hurry. Just wanted to pass it along.

@jchildress The Layout Debugger should be working in 1.5.10-SNAPSHOT now 🙂

@edvin It does. Thank you for fixing.

Thanks for reporting. This whole thing needs a rewrite, it is by far the ugliest piece of code in the framework.

I don't use it too much, but it is nice to have if it's not too hard to maintain. I usually use Scenic View...

Me too, but I aim to add more features to the layout debugger down the line so it's usable for most common cases.

@edvin Chapter 15 (EventBus) Under the "Structure of the EventBus" section in the 4th paragraph there is the following sentence "_In the example above, we have added a static import for the RunOs enum, so that we just write BackgroundThread instead of EventBus.RunOn.BackgroundThread._". The example doesn't use a static import.

and I'm guessing it should be RunOn instead of RunOs

@jchildress Thanks! Corrected, also saw that I mentioned the

runOnFxApplicationThread

which was a boolean param which later became

RunOn

. Fixing that too now.

I also wrote RunAs in at least one place.

Hot tip: You can access the parent inside a builder statement, as it is already connected by the time the builder block runs. I use it in the Hugo Theme Browser to make a Label change padding when it's parent is hovered:

@User uploaded a file: Access parent container inside a builder

There is now also a

Node.paddingAll

var as well as a

paddingAllProperty

so you can write

paddingAll = 5.0

instead of

padding = Insets(5.0, 5.0, 5.0, 5.0)

@jchildress my bad, I moved a few things around to correct some missing come.

gods you dont keep up for one evening and it takes 20 minutes just to read everything that has been happening

I'm very sorry about that, Ron 😁

why do you mind the

init

so much?