<table><tr><td style="">rooty 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/D19077">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/D19077#413826" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D19077#413826</a>, <a href="https://phabricator.kde.org/p/filipf/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@filipf</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>If I just change Text to Labels and remove hardcoded sizes, we won't be able to fit all the text in. We would then have to increase the root height to accommodate the text. This would only give us a good solution for the default font settings (10pt Noto Sans) with no scaling used.</p>

<p>Here's my idea for how to make it work in all conditions (mockup):</p>

<p><a href="https://phabricator.kde.org/F6619405" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F6619405: sddm-kcm-p1.png</a></p>

<ol class="remarkup-list">
<li class="remarkup-list-item">change the root element from a Rectangle to ScrollView -> scaling and big fonts will dramatically increase the labels and we will simply not be able to fit them in</li>
<li class="remarkup-list-item">push this ScrollView's height all the way down so that "Background" (controlled by QWidgets btw) gets pushed as low as it can get -> IDK how to do this without setting height to something huge</li>
<li class="remarkup-list-item">use ColumnLayout instead of GridView -> so that scaling and big fonts don't break the two column layout</li>
</ol></div>
</blockquote>



<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/D19077#413826" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D19077#413826</a>, <a href="https://phabricator.kde.org/p/filipf/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@filipf</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>If I just change Text to Labels and remove hardcoded sizes, we won't be able to fit all the text in. We would then have to increase the root height to accommodate the text. This would only give us a good solution for the default font settings (10pt Noto Sans) with no scaling used.</p>

<p>Here's my idea for how to make it work in all conditions (mockup):</p>

<p><a href="https://phabricator.kde.org/F6619405" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F6619405: sddm-kcm-p1.png</a></p>

<ol class="remarkup-list">
<li class="remarkup-list-item">change the root element from a Rectangle to ScrollView -> scaling and big fonts will dramatically increase the labels and we will simply not be able to fit them in</li>
<li class="remarkup-list-item">push this ScrollView's height all the way down so that "Background" (controlled by QWidgets btw) gets pushed as low as it can get -> IDK how to do this without setting height to something huge</li>
<li class="remarkup-list-item">use ColumnLayout instead of GridView -> so that scaling and big fonts don't break the two column layout</li>
</ol></div>
</blockquote>

<p>nice! i'm gonna try fiddling around with the qt side of things trying to get that "Background:" label to stick out less</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R123 SDDM Configuration Panel (KCM)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D19077">https://phabricator.kde.org/D19077</a></div></div><br /><div><strong>To: </strong>filipf, Plasma, VDG, ngraham<br /><strong>Cc: </strong>rooty, plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>