<table><tr><td style="">meven 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/D25427">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/D25427#648841" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D25427#648841</a>, <a href="https://phabricator.kde.org/p/davidedmundson/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@davidedmundson</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>I'm happy to argue a case of why we should do this and explore compromises, but doing any code changes or reviews before then is not a good use of time.</p></div>
</blockquote>

<p>Let's argue then.</p>

<p>For single screen, there is nothing to argue, I believe, the patch solves the issue : do not downscale.</p>

<p>For multi-screen, my two cents are that we need to cover both the uses cases : user scaled, and pixel precision.</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">pixel precision is needed for use cases such as debugging, "real" pixel screenshot, screenshot apps such as spectacle to allow them cropping anything on the screen without loss, last resort screenshot option.</li>
<li class="remarkup-list-item">user scaled: is needed for kde users to show off how beautiful their desktop is, an overview of , to show multi-screen apps (those are rare I'd say, games mostly), screenshot apps.</li>
</ul>

<p>If that sound reasonable I will add a <tt style="background: #ebebeb; font-size: 13px;">screenshotFullscreenScaled</tt> or a boolean <tt style="background: #ebebeb; font-size: 13px;">scaled</tt> argument to <tt style="background: #ebebeb; font-size: 13px;">screenshotFullscreen</tt>, to the screenshotEffect.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R108 KWin</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D25427">https://phabricator.kde.org/D25427</a></div></div><br /><div><strong>To: </strong>meven, davidedmundson, KWin<br /><strong>Cc: </strong>nicolasfella, zzag, davidre, kwin, Orage, cacarry, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, mkulinski, ragreen, jackyalcine, iodelay, crozbo, bwowk, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, romangg, jensreuterberg, abetts, sebas, apol, ahiemstra, mart<br /></div>