Custom QLayout 'FlowLayout' in QScrollArea, can't update layout

[ComServant]

Okay, I've gotten it working by going like this:

void FlowLayout::insertWidgetAt(QWidget *widget, int index)
{
    widget->setParent(this->parentWidget());
    this->itemList.insert(index, new QWidgetItem(widget));
}

void FlowLayout::insertWidgetAt(QWidget *widget, int index)
{
    widget->setParent(this->parentWidget());
    this->itemList.insert(index, new QWidgetItem(widget));
}

To copy to clipboard, switch view to plain text mode 

So all I needed to do was make sure the widget being added actually knows who its parent is.
I suppose 'QLayout::addWidget()' normally does it, before passing the widget as a item to 'myCustomLayout::addItem()', but since I couldn't find the source for QLayout, I can't be sure.

[Edit:] Found the QLayout source. Apparently the proper and safer call is 'QLayout::addChildWidget()':

void FlowLayout::insertWidgetAt(QWidget *widget, int index)
{
    QLayout::addChildWidget(widget);
    this->itemList.insert(index, new QWidgetItem(widget));
}

void FlowLayout::insertWidgetAt(QWidget *widget, int index)
{
    QLayout::addChildWidget(widget);
    this->itemList.insert(index, new QWidgetItem(widget));
}

To copy to clipboard, switch view to plain text mode 

This is what QLayout::addWidget() calls internally, and it checks for a number of different circumstances making the code more robust.

[norobro]

Your code worked fine after moving the "setLayout" statement.

snapshot1.png

[ComServant]

Cool, but "QLayout::addChildWidget(widget)" is the proper way to do it, (or at least the way QLayout itself does it).
It works fine this way, even after calls to setLayout(), which gives the programmer (me) more flexibility.

It wouldn't be cool to have to call setLayout() after I insert the widgets... the whole point of 'inserting' the widgets, is to add widgets after the layout is already filled and created.