IncrementalSearchBox: fix rendering during terminal scroll
ClosedPublic

Authored by mglb on Wed, Jul 11, 2:10 AM.

Details

Summary

Prevent search widget from moving when scrolling session.

BUG: 396119

Test Plan
  • Run seq 1000
  • Turn on search
  • Hold Shift+Up, Shift+Down

Actual result: the search box scrolls with terminal contents
Expected result: only terminal should scroll, the box should remain
unchanged

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.
mglb created this revision.Wed, Jul 11, 2:10 AM
Restricted Application added a project: Konsole. · View Herald TranscriptWed, Jul 11, 2:10 AM
Restricted Application added a subscriber: konsole-devel. · View Herald Transcript
mglb requested review of this revision.Wed, Jul 11, 2:10 AM
mglb edited the test plan for this revision. (Show Details)Wed, Jul 11, 2:15 AM
mglb added a reviewer: Konsole.

Please add BUG: 396119 since this fixes that bug.

mglb edited the summary of this revision. (Show Details)Wed, Jul 11, 3:34 AM
hindenburg accepted this revision.Wed, Jul 11, 11:24 AM
hindenburg added a subscriber: hindenburg.

very nice - glad this was a simple fix

This revision is now accepted and ready to land.Wed, Jul 11, 11:24 AM

Awesome, now don't forget to run arc amend on your local checkout before landing. :)

hindenburg edited the summary of this revision. (Show Details)Thu, Jul 12, 12:43 AM

I had to add more info in the summary area to get the BUG: to be on a new line instead of summary line.

This revision was automatically updated to reflect the committed changes.

I had to add more info in the summary area to get the BUG: to be on a new line instead of summary line.

That bug in the parser that imposed this restriction has since been fixed, BTW. the BUG: keyword can now be on the first line here in Phabricator.