filipf created this revision.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
filipf requested review of this revision.

REVISION SUMMARY
  This is a fix for D25043 <https://phabricator.kde.org/D25043>.
  
  Using disabledTextColor results in bad text legibility with Plasma themes 
that use filled-style highlight effects.
  
  If colors are prefered over opacity as means of achieving text shading then 
we need to mix disabledTextColor with textColor on selection.
  
  This creates an effect that is comparable to the opacity: isCurrent? 0.8 : 
0.6 code that is used throughout Plasma for subtitles.

REPOSITORY
  R112 Milou

BRANCH
  fix-bad-text-legibility (branched from master)

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

AFFECTED FILES
  lib/qml/ResultDelegate.qml

To: filipf
Cc: 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

Reply via email to