<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/D19255">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/D19255#418729" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D19255#418729</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);"><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>Maybe add a TODO comment on merge to the setLogicalSize (that it does not handle the size plus scale change correctly in every case).</p></blockquote>

<p>Can you expand on what you mean?</p></div>
</blockquote>

<p>I thought we discussed this problem already in the other diff. Example:</p>

<p>Single 4K Monitor is used in 1080p with 1 times scaling: logical size is 1080p<br />
Now an atomic change comes of: Resolution set to 4K, scaling factor to 2. This triggers a wl_output::done event, but the logical size stays the same, i.e. not necessary set the dirty bit.</p>

<p>I just realize with our current KWin/KWayland code, which has to set the logical size on every pixelSize and scale change we do not have this problem at the moment, but later if we want to send only one done event per atomic configuration change a single wl_output::done event might be generated but not the respective xdg_output::done event.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R127 KWayland</div></div></div><br /><div><strong>BRANCH</strong><div><div>master</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D19255">https://phabricator.kde.org/D19255</a></div></div><br /><div><strong>To: </strong>davidedmundson, KWin, romangg<br /><strong>Cc: </strong>romangg, kde-frameworks-devel, michaelh, ngraham, bruns<br /></div>