<table><tr><td style="">kossebau 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/D7061" rel="noreferrer">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/D7061#132805" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: line-through;" rel="noreferrer">D7061#132805</a>, <a href="https://phabricator.kde.org/p/brauch/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;" rel="noreferrer">@brauch</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>Looks good to me -- I think we should try to make the search more discoverable though, IMO in the navigation bar up there it looks like it is a global search (i.e. not on the current page). All the other buttons in that bar are for navigation between pages, just this one is for navigation inside the page.</p></div>
</blockquote>

<p>Agreed. No instance idea how to improve that though.</p>

<p>On Chromium and Firefox the search action is in the action menu and also triggerable by key shortcut.  Given that we cannot have a key shortcut currently, moving the search action into another menu makes it more cumbersome to activate. For a start we should though at least put it also in the view context menu, which BTW seems currently broken, in not being ours. Will look into fixing that menu now at least.</p>

<p>Qt Assistant has the search button on the toolbar as well, but actually positioned away from the navigation tools, and also menuentry-texted and  tooltipped with "Find in Text...". Guess that is something to follow, will give a try.</p></div></div><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D7061#inline-29140" rel="noreferrer">View Inline</a><span style="color: #4b4d51; font-weight: bold;">brauch</span> wrote in <span style="color: #4b4d51; font-weight: bold;">standarddocumentationview.cpp:186</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">a bit unconventional naming, I'd simply go for something like "updateSearchPattern"</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Changed it at least to "searchIncremental" which seems more like commonly used lingo, cmp. <a href="https://en.wikipedia.org/wiki/Incremental_search" class="remarkup-link" target="_blank" rel="noreferrer">https://en.wikipedia.org/wiki/Incremental_search</a></p>

<p style="padding: 0; margin: 8px;">Idea with the naming was to describe the action that is done here. Would agree it still can be improved, but have no clear concept of the whole logic pattern, so lousy at naming, agreed.<br />
Right now I would perhaps name it "updateIncrementalSearch"... perhaps something to improve the next time someone overhauls that code.</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R33 KDevPlatform</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D7061" rel="noreferrer">https://phabricator.kde.org/D7061</a></div></div><br /><div><strong>To: </strong>kossebau, KDevelop, brauch<br /><strong>Cc: </strong>brauch, igorkushnir, kdevelop-devel<br /></div>