<table><tr><td style="">dhaumann 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/D5802" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>I am not yet convinced this is a good idea: We changed the foreground color of highlighted text just recently: <a href="https://git.reviewboard.kde.org/r/127554/diff/2/" class="remarkup-link" target="_blank" rel="noreferrer">https://git.reviewboard.kde.org/r/127554/diff/2/</a></p>

<p>The idea is to have just one color for search results that need to match. This currently is the foreground color of normal text. We would need to really make sure all color schemes still work properly.</p>

<p>As I understand, this is the case?</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R39 KTextEditor</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D5802" rel="noreferrer">https://phabricator.kde.org/D5802</a></div></div><br /><div><strong>To: </strong>intelfx, KDevelop, KTextEditor, Kate<br /><strong>Cc: </strong>dhaumann, mwolff, kwrite-devel, Frameworks<br /></div>