[Konsole-devel] Review Request: Konsole: Switching tabs resets search

[Lindsay Roberts]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/105121/
-----------------------------------------------------------

(Updated June 1, 2012, 7:17 a.m.)


Review request for Konsole.


Description (updated)
-------

Switching to a tab with history search currently produces unexpected results.

This patch makes the following functional changes:

c1) Switching tabs back to a tab with search no longer resets the view to the first top down search result,


Technical issue details:

t1) Switching tab was clearing search state and restarting.
t2) Search position state is based entirely on terminal selection.
t3) Terminal selection is cleared when the window size changes.
t4) The window size changes whenever the search bar is shown/hidden.
t5) The search bar is shown/hidden when swapping between tabs that have differing enablement of the search bar.
t6) Search start position without selection was the most recent line.
t7) Default search direction is (Old -> New).

Combined, every tab switch in most circumstances was resetting the view to the oldest match against the current search bar contents.

Issues fixed:

f1) Switching tabs now only handles re-enable of the search bar, search highlights and buffer position do not change.


Issues unaddressed:

u1) Retaining selection - believe out of scope.
u2) Default search direction. I believe it would be more in line with user expectation to search from newest to oldest by default, but didn't want to see the current patch blocked by controversy.
u3) Per-tab search text - again out of scope.
u4) [Moved out for separate review] Searches with no selection search from the opposite extent of the visible window.


This addresses bug 168769.
    http://bugs.kde.org/show_bug.cgi?id=168769


Diffs
-----

  src/SessionController.h 8c5db37 
  src/SessionController.cpp 0df29a8 

Diff: http://git.reviewboard.kde.org/r/105121/diff/


Testing
-------

Tested all combinations of tabs with and without search bars enabled. Changing search contents before switching back, scrolling, deselection searches.


Thanks,

Lindsay Roberts

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/konsole-devel/attachments/20120601/c39b2272/attachment.html>