D5802: ViewPrivate, KateSearchBar, KateVi::MatchHighlighter: use selection foreground for search highlights

[Ivan Shapovalov]

intelfx added a comment.


  @mwolff @dhaumann @brauch I'm sorry for my silence — I stopped using KDE/KDevelop some time ago and did not track this changeset properly. Could you please reiterate what was the conclusion and what were @mwolff's requested changes?
  
  > 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.
  
  Sorry, I did not understand this. Could you please rephrase?
  
  > Couldn't this be fixed by checking what color has the higher contrast - selection or normal? Then pick the one that has the higher contrast.
  
  IMO this is too specific logic to hard-code. What if there is a schema explicitly designed for low contrast?
  
  > Can't we simply update our shipped schemas, and expect users with custom schemas to fix them?
  
  I believe many would oppose :)

REPOSITORY
  R39 KTextEditor

REVISION DETAIL
  https://phabricator.kde.org/D5802

To: intelfx, #kdevelop, #ktexteditor, #kate, mwolff
Cc: kde-frameworks-devel, brauch, dhaumann, mwolff, kwrite-devel, michaelh, kevinapavew, ngraham, bruns, demsking, cullmann, sars
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwrite-devel/attachments/20180815/332a1766/attachment.html>