<table><tr><td style="">loh.tar 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/D17602">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>I'm not sure why you have to move the setFocus() call to the beginning of the function... your comment says something about selection...</p></blockquote>

<p>I move this up to reduce code redundancies. Without this patch was only the focus changed when we hit the right item, now happens this almost every time. But in these rare case would then the chosen item kept the focus, but that hurts literally my eyes.</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>I don't think we need the buttons for next/previous. If you use the keyboard for navigating you want the shortcuts and if you use the mouse it is much quicker to just navigate the results and click the wanted item.</p></blockquote>

<p>I guess I didn't need these buttons too, but have the feeling they should be there</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>I don't think we are pressed on space to need to merge the buttons...</p></blockquote>

<p>Well, I like such behaviour. So the space is not the reason :-)</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>Changing the next/previous slots to go to the items with line numbers would be a good change.</p></blockquote>

<p>Good, will try to do it (here or extra patch?)</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R40 Kate</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D17602">https://phabricator.kde.org/D17602</a></div></div><br /><div><strong>To: </strong>loh.tar, Kate, sars<br /><strong>Cc: </strong>kwrite-devel, Kate, hase, michaelh, ngraham, demsking, cullmann, sars, dhaumann<br /></div>