<table><tr><td style="">wbauer created this revision.<br />wbauer added a reviewer: Konsole.<br />wbauer added a project: Konsole.<br />Herald added a subscriber: konsole-devel.<br />wbauer requested review of this revision.
</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/D19513">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Since anti-aliasing was enabled in the painter, coordinates need to be shifted half a pixel so that they align with the pixel grid, otherwise the result gets "blurred" due to the anti-aliasing.<br />
And as parts of the blurred shape leak outside the cursor rectangle, this also leaves artifacts when the cursor moves as these parts are not cleared.</p>

<p>This is basically the same as <a href="https://phabricator.kde.org/R319:e7085310d6d594823d0ed491fa8bdbd99dec4932" class="remarkup-link" target="_blank" rel="noreferrer">https://phabricator.kde.org/R319:e7085310d6d594823d0ed491fa8bdbd99dec4932</a> for the standard block cursor.</p>

<p>BUG: 402589</p></div></div><br /><div><strong>TEST PLAN</strong><div><ul class="remarkup-list">
<li class="remarkup-list-item">Switch cursor shape to "I-Beam" or "Underline" in the "Advanced" profile settings</li>
</ul>

<p>The cursors are a single line again now, before they were blurred by anti-aliasing.</p>

<p>Screenshots:<br />
Before:<br />
<a href="https://phabricator.kde.org/F6656366" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F6656366: ibeam-before.png</a><br />
<a href="https://phabricator.kde.org/F6656370" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F6656370: underline-before.png</a></p>

<p>After:<br />
<a href="https://phabricator.kde.org/F6656371" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F6656371: ibeam-after.png</a><br />
<a href="https://phabricator.kde.org/F6656373" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F6656373: underline-after.png</a></p>

<p>Also, there are no more artifacts when the cursor is moved.</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/D19513">https://phabricator.kde.org/D19513</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/TerminalDisplay.cpp</div></div></div><br /><div><strong>To: </strong>wbauer, Konsole<br /><strong>Cc: </strong>konsole-devel, gennad, maciejn, thsurrel, ngraham, maximilianocuria, hindenburg<br /></div>