<table><tr><td style="">filipf 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/D21668">View Revision</a></tr></table><br /><div><div><p>Thanks for having a look guys :)</p>

<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/D21668#478879" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D21668#478879</a>, <a href="https://phabricator.kde.org/p/ngraham/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@ngraham</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>Much better! Is there a reason why we need a ColumnLayout to hold those two FormLayouts in <tt style="background: #ebebeb; font-size: 13px;">main.qml</tt>? Could all the stuff in the second one just go into the first one instead? Then you could get rid of the Columnlayout entirely</p></div>
</blockquote>

<p>That's what I wanted to do in the first place and was the first version I posted here, but it resulted in big binding loop issues and twinFormLayouts of loader forms failing to achieve alignment with the main one.</p>

<p>There's still some points of improvement here I think:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">child form layouts are only interlinked with the main one, but not vice versa -> the odds aren't big, but should there be any language where the labels in child layouts are longer that the ones in the main one, it will mess up the alignment</li>
<li class="remarkup-list-item">old bug: the times in the Locations mode should automatically be updated upon entering new coordinates (right now it only does so when switching modes)</li>
<li class="remarkup-list-item">kcm doesn't have the totally left aligned form layout mode when you shrink the window -> might be because of the SimpleKCM wrapper, not sure</li>
</ul>

<p>Fixing those things might come a lot easier to someone with more experience, but I'll keep them in mind for when I have more free time.</p>

<p>For now I'll land this as I think the UI is in dire need of improvement!</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</div></div></div><br /><div><strong>BRANCH</strong><div><div>modernize-redesign-nightcolor-config (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D21668">https://phabricator.kde.org/D21668</a></div></div><br /><div><strong>To: </strong>filipf, VDG, ngraham, romangg, mart, Plasma, davidedmundson<br /><strong>Cc: </strong>davidedmundson, plasma-devel, LeGast00n, ericadams, jraleigh, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>