<table><tr><td style="">sandsmark added a comment.
</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/D17744">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D17744#381034" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D17744#381034</a>, <a href="https://phabricator.kde.org/p/mglb/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@mglb</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><ol class="remarkup-list">
<li class="remarkup-list-item"><ul class="remarkup-list">
<li class="remarkup-list-item">Invoke search, type text: all nice, I see current search line and remaining search results</li>
<li class="remarkup-list-item">Close search</li>
<li class="remarkup-list-item">Invoke search again: There is no current line (this is OK) and no dim (not OK) - search results are barely visible (without transparency they are not visible at all)</li>
</ul></li>
</ol></div>
</blockquote>

<p>Fixed. But because we don't have any other way of indicating whether the search is active other than setting the current result line being valid I solved it by preserving the current search line (which makes more sense anyhow, imho).</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><ol class="remarkup-list" start="2">
<li class="remarkup-list-item">Font size change (Ctrl+Scroll, Ctrl++, Ctrl+-) during search is not handled</li>
</ol></blockquote>

<p>Not sure what you mean, it seems to work here.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>Pedantic things:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Selecting search result removes the rounded rect's rounding <a href="https://phabricator.kde.org/F6502287" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F6502287: rounded-corners.png</a></li>
</ul></blockquote>

<p>I explicitly didn't change this, to indicate clearly that it is selected, even when searching.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><ul class="remarkup-list">
<li class="remarkup-list-item">What do you think about making the lines 1px tall, drawn on inner edge of the rectangle? This would match lines in Breeze widget style.</li>
</ul></blockquote>

<p>Not entirely sure what you mean.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><ul class="remarkup-list">
<li class="remarkup-list-item">Configurable colors, or colors taken from system theme/konsole theme</li>
</ul></blockquote>

<p>Tested this a bit first, but it's hard to make it work with all kinds of different application (<tt style="background: #ebebeb; font-size: 13px;">make menuconfig</tt> is what broke it for me, but mc etc. is also good at breaking this).</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R319 Konsole</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D17744">https://phabricator.kde.org/D17744</a></div></div><br /><div><strong>To: </strong>sandsmark, hindenburg, Konsole, thsurrel, tcanabrava, VDG<br /><strong>Cc: </strong>ngraham, mglb, konsole-devel, Konsole, thsurrel, maximilianocuria, hindenburg<br /></div>