D25043: [ResultDelegate] Use theme.disabledColor

[Noah Davis]

ndavis added a comment.


  I think I agree with @filipf that disabledTextColor should not be used here. disabledTextColor is also semantically incorrect. This seems to assume that the disabled text color will always be the correct shade of gray for this situation and that's not necessarily true. Is there a way to use something like `KColorUtils::mix(textColor, windowColor, 0.2)` in QML? That would allow you to get a color similar to reducing opacity without actually making the color semi-transparent, which improves readability.

REPOSITORY
  R112 Milou

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

To: broulik, #plasma, #vdg, davidedmundson
Cc: ndavis, filipf, davidedmundson, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20191102/da824812/attachment.html>