TerraFormSkin is currently doing the right thing, modifying row height based on the relative baselines of the label and field.
Cool.
But even the way that TerraFormSkin uses the baselines, it is not really making use of the width argument,
In a sense, it is, because some components determine their baselines based on the width value. Internally, they calculate a preferred height given the width constraint and then report the baseline as relative to that height. That's why I think we should replace the width argument with a height argument - this type of implementation isn't guaranteed to work 100% of the time, because components are not guaranteed to be given their preferred height.
I think the best thing to do would be to remove the width argument and leave it like that until we have a clear use case for needing something more.
I actually think we should make this change now. If we don't, it will require an API change later. Since it is simple to do and we know we're going to want to do it later, we may as well do it now. We don't necessarily need to update FlowPane to use it right away, but at least the feature will be available when we are ready to do so.
G
