yeah, that's not quite what i want

forms seem to be pretty inherently VBox based

no biggie

@User uploaded a file:

@User uploaded a file: Untitled

so I think you can have a second label

thanks

not like the code you are making but it does show that multiple labels are possible

@ron - just to be clear - that will not honour the horizontal/vertical positioning of labels for the field (i.e. you would need to keep the two in sync in your field definition)

yes that is correct

@edvin - do you think this would be a good feature?

@edvin I think its a good feature 🙂 I would use it

I will probably have some time this weekend to look at it, but could use a bit of guidance in terms of what the best approach might be. I realize this stuff has to play nice with the responsive layout functionality as well..

So basically we would like to have rows inside a fieldset? That wouldn't be so hard 🙂

It could also be solved with multiple fieldsets next to each other.

Actually, both of those features would be helpful to create more complex layouts, but I think a logical row inside a fieldset would solve @mikehearn's usecase.

Success 🙂 Today's snapshot was published to sonatype automatically 🙂

@mikehearn Very cool about the go-style app binary. Let us know when you have something you can show us :))

I'm investigating some solutions to the forms issue now.

Just to be clear, the reason that Mike's layout attempt failed, is that the

field

builder is only available on

fieldset

, so even if you create those vboxes and hboxes, the fields are only added to the fieldset directly. That's why the layout doesn't change.

The reason for this, is that a

Field

expects it's parent to be a

Fieldset

.

Many things in

Field

,

Fieldset

and

Form

expect direct children. We could rewrite that, or make

Fieldset

into a

Pane

and do the layout ourselves there. The most intuitive for users might be to support any kind of layout inside a fieldset. Let me give that a shot after dinner.

There. Both dinner and rewrite is done.

@mikehearn I have refactored the Form layout so it supports arbitrary layout containers inside a

Fieldset

. Your original code will now work as expected, though you might want to add some spacing to the surrounding hbox. For example

hbox(20) { .. }

.

@mikehearn Would you like a snapshot build published to sonatype perhaps?

@mikehearn A code style tip:

field("Node Name", op = { nodeNameField() })

can be written as

field("Node Name") { nodeNameField() }

. Looks cleaner. I normally also recommend creating the field inside the builder, but that's a matter of preference I guess.

@User uploaded a file:

@User uploaded a file: Untitled

By specifying vertical label position via

fieldset("FieldSet", labelPosition = Orientation.VERTICAL)

you get: