<table><tr><td style="">ngraham 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><p>This introduces a new warning:</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);">WARNING: viewBackgroundColor is deprecated, use backgroundColor with colorSet: Theme.View instead</pre></div>

<p>+1 on removing the italic styling. I also think bolding the title is unnecessary too while it's using its maximum size (level 1). Bolding it only seems necessary when it's closer to the size of the text below it.</p>

<p>I still don't like how the ColumnLayout of information causes the bottom-most items to get cut off with the default System Settings window height. I think we could consolidate some of the information, like moving the license onto one of the other lines.</p>

<p>Here's basically how it would look:<br />
<a href="https://phabricator.kde.org/F6628909" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F6628909: Screenshot_20190221_160448.png</a></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>GB_2, mmustac, davidedmundson, abetts, rooty, plasma-devel, jraleigh, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, sebas, apol, mart<br /></div>