Sweet. It might be cleaner to use CSS to set the look though:

@User uploaded a file: Untitled

We could use the oppertunity to apply a local stylesheet even..

But no.. better to keep it clean and create something separate for that.

You could of course add the styles to the "global" stylesheet:

@User uploaded a file: Untitled

And then just do

addClass(Styles.displayBox)

inside the

vbox

builder. Then you don't need to muck up the sample with look and feel code at all.

@User uploaded a file: Demoing alignment and commented: @edvin factored out vbox attributes

that syntactically works, but it gave me very different results

…the it.style construct

It put the drop shadow on the Labels and the VBoxes didn’t get a pref size

Also, is this the correct way to set a margin within the vbox? This is working for me, but it uses the vboxConstraints

vbox { label(Pos.TOP_LEFT.toString()) { vboxConstraints { margin = Insets(10.0) } } alignment = Pos.TOP_LEFT }

You can do this just below the last vbox. Then

children

will be the list of vboxes, and you don't need init, with(root) and val tp. You also don't need parenthesis before the block in the forEach. You might also do `with(it as VBox) { setAttributesHere() }. How it can set the dropshadow on the label I can't even understand, since you explicitly set it on the vbox. I also tried this with "my" syntax earlier and it worked with exactly the same visual result.

This is probably how it should look:

@User uploaded a file: Untitled

@User uploaded a file:

This looks clean to me. Should I commit it?

I see no reason to set the

background

property when you only override the background color, so

backgroundColor += Color.TAN

might be less confusing, and much easier on the eye.

Using CSS is generally nicer than setting properties on the node itself IMO.

Before you commit, can you put a margin on the Labels? I have this

@User uploaded a file: Untitled

which is back to the vboxConstraints. I completely agree with the CSS. I put as much in CSS as possible for my clients.

OK, but we don't need the

vboxConstraints

anymore, we can just use

margin

directly.

I also added a small trick so you don't need to iterate all nodes in the children list to apply styles:

@User uploaded a file: Apply styles to all children

10 px margin was actually a bit much, I set it to 5. Commited now, and deployed the kitchensink 🙂

@User uploaded a file:

I also update the readme a little 🙂

Great. Was the app structure ok? Samples.json, etc?