D20766: Use appropriate background color for text previews

Volker Krause noreply at phabricator.kde.org
Tue Apr 23 15:20:35 BST 2019


vkrause added subscribers: cullmann, dhaumann.
vkrause added a comment.


  In D20766#454891 <https://phabricator.kde.org/D20766#454891>, @kossebau wrote:
  
  > So, switching also the Plasma Theme to "Breeze Dark", I started to be able to reproduce: seems that the default text color is influenced by that. Which seems a side-effect not to be expected. And this patch here just a work-around for the symptom. We ask KSyntaxHighlighting::Repository for a light theme, we do expect one.
  >  IMHO this seems rather a bug in KSyntaxHighlighting and should be fixed there.
  
  
  @dhaumann @cullmann  ^ this sounds like we are making the assumption that the current system color theme from the palette matches the syntax highlighting theme, ie. the palette provided text color is compatible with the selected theme. That's probably true in the common editor case, but it seems to be an issue here. Should the syntax highlighting themes always have a hardcoded text color? Doesn't seem entirely clean either, as following the system color makes sense for different color schemes with the same "lightness"/"darkness". Or should we try to detect insufficient contrast between foreground and background, and use an alternate color in that case?

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

To: eshalygin, kossebau, cfeck
Cc: dhaumann, cullmann, vkrause, cfeck, meven, broulik, kde-frameworks-devel, kfm-devel, alexde, feverfew, michaelh, spoorun, navarromorales, firef, ngraham, andrebarros, bruns, emmanuelp, mikesomov
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20190423/0f0e79e7/attachment.htm>


More information about the kfm-devel mailing list