<table><tr><td style="">aacid accepted this revision.<br />aacid added a comment.<br />This revision is now accepted and ready to land.
</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/D7662">View Revision</a></tr></table><br /><div><div><p>That is a fix, not the ideal fix, now that i've had time to look at the problem, the problem is.</p>

<p>Basically that we're writting in the "buffered" area of the painter, which means we reuse it from paint to paint unless something big has happened, since we reuse it from paint to paint and if we're intersecting with limits it means that when a highlight it is cut by border "of the screen" we'll only paint part of the highlight, which means that when we scroll to show all the higlight we only paint the "new" area of the highlight, reusing the old one, that was fine before since it's just a colored rectangle, but now that it has borders it means that the old border that was "in the middle" of the highlight is kept.</p>

<p>Ideally one would move the border painting to the non buffered area (i.e. the area that is not reused from paint to paint), but that'd be quite a lot of job so what i've made is partially use the suggestion from <a href="https://phabricator.kde.org/p/simgunz/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@simgunz</a> but only applied to the hightlight rects.</p>

<p>I'll commit in a minute</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R223 Okular</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D7662">https://phabricator.kde.org/D7662</a></div></div><br /><div><strong>To: </strong>sander, Okular, aacid<br /><strong>Cc: </strong>okular-devel, simgunz, cfeck, aacid, ngraham<br /></div>