layout->addSpacerItem(new QSpacerItem(20,20,QSizePolicy::Preferred, QSizePolicy::MinimumExpanding));

This will keep the spacing from expanding between the lines as the widget get taller.

I was able to get the data with in 175ms. With a wait that low the widget does it quick enough were you don't notice it was ever empty. If we are going to wait a full 2 seconds can we provide some indication here we are fetching the news?

Changed to 500ms. 200ms for me wasn't enough. I think that time will depend on the internet. Will add a fallback

Do we want to see the fallback if one fails?

This button is really Small and not in line with the text.

With the refresh button I would consider making this just one line
"Failed to fetch News <retry button>"

idk why that came out so small. But update the phrase

This is causing the crash

Remove all this code.

Make this new private function

void WelcomeWidget::setNewsLayout(QLayout *newLayout)
{
    if (m_newsFeedWidget->layout()) {
        qDeleteAll(m_newsFeedWidget->children());
    }
    m_newsFeedWidget->setLayout(newLayout);
}

Use that here to give the new Widget its layout
and Again in the fallback to give its layout.

no more crash no more double layouts.

Good Start PushButtons are ok but we should use a Tool Button here its really made for just icons. we also set the icon size to fontMetrics().lineSpacing() this should ensure out button take up all the lines height no matter the font size.

auto button = new QToolButton;
    button->setIcon(QIcon::fromTheme("view-refresh", QIcon(QString(":/%1/refresh").arg(theme))));
    button->setIconSize(QSize(fontMetrics().lineSpacing(),fontMetrics().lineSpacing()));

idk about the iconSize, because if the user gets a big font the icon will be bigger than the defaults. at least is my impression.
@tcanabrava any thoughts?

missed one
setNewsLayout(layout);

Const ref

Ref

Missing this as 3rd argument of connect

Print error message

C++ for each

Second arg not necessary

Qstringliteral

Alphabetic

Alphabetic order