D20766: Use appropriate background color for text previews

Méven Car noreply at phabricator.kde.org
Thu Nov 14 18:00:26 GMT 2019


meven added a comment.


  I believe the issue currently is that for text files or files without associated definition we end up with an invalid definition that prevents SyntaxHighlighter::setDefinition to call rehighlight().
  QTextDocument uses then whatever it get from the theme I guess.
  Kate has an explicit special case in KateHighlighting::KateHighlighting to insert a default Format.
  
  I think either KSyntaxHighlighting should have a dummy default definition based on currrent theme or we should here create a dummy default definition when `m_highlightingRepository.definitionForFileName(path)` return an invalid definition.
  What do you think ?

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

To: eshalygin, kossebau, cfeck
Cc: dhaumann, cullmann, vkrause, cfeck, meven, broulik, kde-frameworks-devel, kfm-devel, pberestov, iasensio, fprice, LeGast00n, MrPepe, fbampaloukas, alexde, GB_2, Codezela, 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/20191114/267b3b65/attachment.htm>


More information about the kfm-devel mailing list