<table><tr><td style="">antonanikin 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/D14288">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/D14288#296131" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D14288#296131</a>, <a href="https://phabricator.kde.org/p/kossebau/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@kossebau</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>And one more request (from me personally, no idea about current kdevelop maintainers' POV): could you consider doing the UI in ui files?</p>

<p>Myself used to write all QWidget UI setup also directly in code, though big projects like also KDevelop have teached me that it improves maintainability if concerns are separated where possible, including moving the declaration of (static) UI into the prepared infrastructure, i.e. the UI files. While it adds to build time, it does not add that much to runtime/codesize, if at all.</p>

<p>And even if done for consistency with code of (most) existing plugins :)</p></div>
</blockquote>

<p>Ok. I will convert <tt style="background: #ebebeb; font-size: 13px;">GlobalConfigPage</tt> and <tt style="background: #ebebeb; font-size: 13px;">ProjectConfigPage</tt> into <tt style="background: #ebebeb; font-size: 13px;">.ui</tt>-based widgets. I also prefer "hand-written" UI but you are right about better maintainability of <tt style="background: #ebebeb; font-size: 13px;">.ui</tt> files for big community projects.</p>

<p><tt style="background: #ebebeb; font-size: 13px;">ChecksWidget</tt> is special non-static widget for <tt style="background: #ebebeb; font-size: 13px;">KConfig</tt> so I think it should not be changed. What are you think about <tt style="background: #ebebeb; font-size: 13px;">CommandLineWidget</tt>? It simple but I also can convert it into <tt style="background: #ebebeb; font-size: 13px;">.ui</tt>-form if this will be better in maintainability perspective.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R32 KDevelop</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D14288">https://phabricator.kde.org/D14288</a></div></div><br /><div><strong>To: </strong>antonanikin, KDevelop<br /><strong>Cc: </strong>kossebau, kdevelop-devel, antismap, iodelay, vbspam, njensen, geetamc, Pilzschaf, akshaydeo, surgenight, arrowd<br /></div>