Add BBox support to BLayoutBuilder

[pulkomandy]

It would be nice to be able to do this:

BLayoutBuilder::Group<>(someView)
    .Add(...)
    .AddBox("box label")
        .AddGroup(B_VERTICAL)
            ...
        .End()
    .End()
.End();

Currently, using BBox in layout mode is annoying, you have to:

• Declare and allocate the BBox
• Set its label
• Add an inner view to it
• Use a BLayoutBuilder to populate that inner view
• Finally add the box to the global layout

This breaks the nice pseudo-declarative workflow for GUIs.

[X512]

Can you give an equivalent code based on current existing API?

[pulkomandy]

With current API you have to do something like this:

BBox* box = new BBox("box label");
BGroupView* innerView = new BGroupView(B_VERTICAL);
box->AddChild(innerView);

BLayoutBuilder::Group<>(innerView)
    ... // Same content as the ... in the initial example
.End();

BLayoutBuilder::Group<>(someView)
    .Add(...)
    .Add("box")
.End();

[cafeina]

Would it make sense to also have a template like this:

BLayoutBuilder::Box<>("Title", ...)

.Add(...)

.End();

[pulkomandy]

Yes, for completeness and consistency with the other layouts, and it may be required by the implementation of AddBox in the other layouts as well (that will return such a type)