<table><tr><td style="">kossebau added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D4413" rel="noreferrer">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D4413#82710" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">D4413#82710</a>, <a href="https://phabricator.kde.org/p/mwolff/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;" rel="noreferrer">@mwolff</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>this is really dirty. can you please change the code to only add stuff to a "known" widget and then delete that widget and its children here?</p></div>
</blockquote>

<p>Hm, would that be less dirty? Any solution is fragile as they all require that any created QWidget does not get the page itself set as parent widget, otherwise it would be missed out in any cleanup.<br />
I initially did a separate list of any created widgets, though explicit and not as proposed here, implicit by adding another dummy QWidget,. But then I realized that all the information is there in the existing data structure and just needs to be queried when needed. Which is what the loop did here.</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">delete m_containerWidget;
m_containerWidget = new QWidget(this);
layout()->addWidget(m_containerWidget);</pre></div>

<p>would be what you prefer here?<br />
On debugging UI structures I personall dislike any unneeded extra QWidget, Thus proposing the explicit collection of the groupbox widgets then, as compromise. Though I still think that initially proposed loop is good and just looks dirty because it had so many explanations ;)</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R33 KDevPlatform</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D4413" rel="noreferrer">https://phabricator.kde.org/D4413</a></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>kossebau, mwolff, KDevelop<br /><strong>Cc: </strong>mwolff, kdevelop-devel<br /></div>