[Terminal display] Limit the search result line highlight width
ClosedPublic

Authored by thsurrel on Nov 7 2018, 9:47 AM.

Details

Summary

The search result highlight was using the whole container width,
this creates some glitches when scrolling because the right-most
part of the line is not repainted.
Limiting the search highlight to the number of columns fixes this
and is more consistent with line selection highlight too.

Test Plan

Highlight a line by searching something in the konsole, then scroll.
There should be no glitches left on the right.

Diff Detail

Repository
R319 Konsole
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
thsurrel created this revision.Nov 7 2018, 9:47 AM
Restricted Application added a project: Konsole. · View Herald TranscriptNov 7 2018, 9:47 AM
Restricted Application added a subscriber: konsole-devel. · View Herald Transcript
thsurrel requested review of this revision.Nov 7 2018, 9:47 AM
thsurrel edited the test plan for this revision. (Show Details)Nov 7 2018, 9:47 AM

Thanks, this fixes it for me. +1

hindenburg accepted this revision.Nov 8 2018, 1:44 AM
hindenburg added a subscriber: hindenburg.

Looks fine thanks - unrelated to this, but i notice when closing the search box, the old highlight lines are not removed when hitting ESC sometimes.

This revision is now accepted and ready to land.Nov 8 2018, 1:44 AM
This revision was automatically updated to reflect the committed changes.

Looks fine thanks - unrelated to this, but i notice when closing the search box, the old highlight lines are not removed when hitting ESC sometimes.

Do you know how to reproduce that ? I could try to fix this, but I have not been able to reproduce it.

Found it, proposed fix in https://phabricator.kde.org/D16754, but I think there is a problem with what I tried to do there ...