<table><tr><td style="">tcanabrava 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/D13174">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/D13174#275232" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D13174#275232</a>, <a href="https://phabricator.kde.org/p/ahmadsamir/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@ahmadsamir</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>One corner case comes to mind (whether it has to be fixed here or in a separate review later on), if the matched text is under the search bar, the user won't be able to see it, this can be further split into two cases (and two different solutions?):<br />
 a) if there's room to scroll up/down to show the text<br />
 b) if there's no room to scroll.</p>

<p>The way chrome handles this is move the search bar to the right/left to show the search result, then move it back to its original location if the user goes to the next search result (if that result isn't under the search bar). (IMHO, this jumping around is a bit bothersome).</p></div>
</blockquote>

<p>I think this is less problematic than what currently happens if you use "top"  open and close the search.</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/D13174">https://phabricator.kde.org/D13174</a></div></div><br /><div><strong>To: </strong>tcanabrava, Konsole, hindenburg<br /><strong>Cc: </strong>ahmadsamir, rizzitello, ngraham, konsole-devel, herrold, maximilianocuria, hindenburg<br /></div>