<table><tr><td style="">ahmadsamir 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/D9008" rel="noreferrer">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/D9008#172541" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">D9008#172541</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;" rel="noreferrer">@ngraham</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>Since this will eventually be fixed, can you conditionalize the workaround like we did in <a href="https://phabricator.kde.org/D8387" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: line-through;" rel="noreferrer">D8387</a>?</p></div>
</blockquote>

<p>To make the fix conditional I think we'd better wait until it's actually fixed upstream, so that we have the actual Qt version with the fix. And hindenburg hasn't accepted the patch yet anyway :)</p>

<p>But if the code path that uses pixelDelta in ScrollState in konsole is supposed to handle high resolution touchpads/trackpads/... etc then checking that the event source is not a mouse can just stay in, even after the bug is fixed upstream in Qt. (I am not sure, but is that code supposed to handle high resolution scroll events coming from say an Apple Magic mouse?).</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>To my knowledge, Scrollviews in Kirigami and Kickoff are also affected in the same way. Would you be able to consider preparing similar patches for them, too?</p></blockquote>

<p>I'll look into that, but no promises though.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R319 Konsole</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D9008" rel="noreferrer">https://phabricator.kde.org/D9008</a></div></div><br /><div><strong>To: </strong>ahmadsamir, Konsole, hindenburg<br /><strong>Cc: </strong>ngraham, hindenburg<br /></div>