<html>
 <body>
  <div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
   <table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
    <tr>
     <td>
      This is an automatically generated e-mail. To reply, visit:
      <a href="http://git.reviewboard.kde.org/r/111766/">http://git.reviewboard.kde.org/r/111766/</a>
     </td>
    </tr>
   </table>
   <br />













<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
 <p style="margin-top: 0;">On August 21st, 2013, 3:15 a.m. UTC, <b>Kurt Hindenburg</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">A few minor code issues which can be fixed later if necessary.  Let me double-check the code and do some testing.  Likely we should break this up into smaller patches but I'm OK with it.

One thing that bothers me is that w/ the search buttons, my terminal smallest width is 56 chars (depending on font size) mainly due to the 'Search from bottom'.  Also, if you have a 40 char width and toggle the search, it will increase the width of the terminal window. With the current code it is 41 characters.  I don't really like this but not sure how to shorten the buttons.</pre>
 </blockquote>







</blockquote>

<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I have not spent much time on code style and commenting yet, just doing the style i'm used to from work. Once you're ok with the functional changes, I'll clean it up before we look at merging.
As you know there has been a lot of back and forth with many commits depending on each other, so most of the commit history is not of much use at this point. I will see what I'm able to reorder, squash or split up.</pre>
<br />


<p>- Harald</p>


<br />
<p>On August 17th, 2013, 8:22 a.m. UTC, Harald Hvaal wrote:</p>








<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
 <tr>
  <td>

<div>Review request for Konsole and Kurt Hindenburg.</div>
<div>By Harald Hvaal.</div>


<p style="color: grey;"><i>Updated Aug. 17, 2013, 8:22 a.m.</i></p>






<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
 <table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
 <tr>
  <td>
   <pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">commit 4f3ac690828c4d7a707ddae38d977798a6e2c13d
Author: Harald Hvaal <harald.hvaal@gmail.com>
Date:   Sun Jul 28 11:24:37 2013

    Various search-related improvements
    
    - Add "search from beginning" feature
        This will scroll to the top and search from there.
        Ctrl+return is also bound to this action
    - Text Highlighted by mouse will be set as the current search text
    - Add "Search backwards" to search bar options

commit 36e59043ba41b4ea9fd31bd961b0b88e20da10e1
Author: Harald Hvaal <harald.hvaal@gmail.com>
Date:   Sat Jul 27 10:07:21 2013

    When showing the search bar, do not invoke a search, only apply the highlight filters

commit fb2202fd4f54d0e1a82a861d0d998582cc08780c
Author: Harald Hvaal <harald.hvaal@gmail.com>
Date:   Sat Jul 27 19:51:32 2013

    Scroll with the result centered

commit 7b8f805fd15e18bb6cdba520292dd7afb0f7e4e3
Author: Harald Hvaal <harald.hvaal@gmail.com>
Date:   Sat Jul 27 09:39:27 2013

    Do not automatically reset the search start line on search hits
    
    This was causing the annoying behavior that if you were to pause while typing
    in a search term, and it would actually find a hit, then you would be searching
    for the term a second time once you finish typing.
    
    Example console output:
    
     ***
    1 usb
    2 hdmi
    3 usb
    4 hdmi
    5 usb
     ***
    
    if you were to search for this by quickly typing "usb" you would get the hit on
    line 1.  If you type "us", wait a moment, then type "b", you will end up on
    line 3. When searching through large console output, this is frustrating as you
    would never really be sure whether you are at the first search result without
    double-checking.
    
    This commit introduces two new behaviors:
    1. When you show the search bar, all searching will be done from the first
        visible line in the terminal.
    2. This start position is only reset when you
        advance to the next result, by pressing "next", "previous", or the shortcuts
        RETURN or SHIFT-RETURN
    
    In the example above, this would ensure that you would end up at the hit on
    line 1 as expected.</pre>
  </td>
 </tr>
</table>





<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">

 <li>src/TerminalDisplay.cpp <span style="color: grey">(acfddc176bc68ae9b0f3a9ea52c958e37d14dff8)</span></li>

 <li>src/TerminalDisplay.h <span style="color: grey">(100ffccc3f8210d5f07d9d24aa14032766415205)</span></li>

 <li>src/SessionController.cpp <span style="color: grey">(88fc50c6a164fed94eefc72a71e4460f15ede57e)</span></li>

 <li>src/SessionController.h <span style="color: grey">(036e0534960fe7a1bd32be04136ba162c0469413)</span></li>

 <li>src/ScreenWindow.cpp <span style="color: grey">(92804302c7df6658a10ee0b8000eae9ec5617df0)</span></li>

 <li>src/ScreenWindow.h <span style="color: grey">(89fe7bed73b41749fae03f24c1868aee62369b18)</span></li>

 <li>src/IncrementalSearchBar.cpp <span style="color: grey">(79a28773c64513ccc95375eef9a51b0e13b81a9f)</span></li>

 <li>src/IncrementalSearchBar.h <span style="color: grey">(d0661a5ae539fe6e11800e2367d6a55660b3028b)</span></li>

</ul>

<p><a href="http://git.reviewboard.kde.org/r/111766/diff/" style="margin-left: 3em;">View Diff</a></p>







  </td>
 </tr>
</table>








  </div>
 </body>
</html>