<table><tr><td style="">romangg 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/D23798">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/D23798#532963" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D23798#532963</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>Need a hand with the <tt style="background: #ebebeb; font-size: 13px;">TODO:</tt> in main.qml.</p></div>
</blockquote>

<p>With can't be really fixed since the config and by that the output model is received asynchronously later on and to my knowledge the KCM's implicitHeight must be set directly in the beginning.</p>

<p>To be honest I'm no friend of hiding the preview/overview anyway when there is only one output connected. Reasons are that people are used to seeing it and that if a second monitor is connected/removed suddenly some huge ass control pattern gets added/removed. This will also always compromise the current window size when connecting a second monitor since it will be too small. Also imo it brings value to have a miniature depiction of the output setup even if there is only one. Instead we could think of making the monitor in the overview not movable when only one monitor is connected.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R104 KScreen</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D23798">https://phabricator.kde.org/D23798</a></div></div><br /><div><strong>To: </strong>ngraham, romangg, VDG, KWin<br /><strong>Cc: </strong>filipf, ndavis, GB_2, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, ragreen, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>