kossebau added a subscriber: cfeck. kossebau added a comment.
Thanks for the patch. Hm, somehow I remember from my testing for D19432 <https://phabricator.kde.org/D19432> that highlighting color did not take active UI color theme into account, has that changed meanwhile? In any case, IMHO there should be first a decision whether thumbnails for plain text documents should be done taking the UI color theme into account, or should be similar to pdf, doc & Co, where usually a print out on paper is assumed for the look (at least by the native formats). Personally I would favour this, and the one doing the hard-coded background color (@cfeck) seems at one point done to have as well. REVISION DETAIL https://phabricator.kde.org/D20766 To: eshalygin, kossebau Cc: cfeck, meven, broulik, kde-frameworks-devel, kfm-devel, alexde, feverfew, michaelh, spoorun, navarromorales, firef, ngraham, andrebarros, bruns, emmanuelp, mikesomov
