Review Request 127637: bind highlightedText color in Plasma::Theme

Marco Martin notmart at gmail.com
Tue Apr 12 13:41:33 UTC 2016


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/127637/
-----------------------------------------------------------

Review request for Plasma.


Repository: plasma-framework


Description
-------

we have a color in the Plasma theme "highlight" that is basically an accent color used around.. but one of the uses is to use it as a background, such as highlighted text. in that case we don't know what color would be safe for contrasting on top of it for things like the highlighted text color (plasma style used the background color for that, which is incorrect)


Diffs
-----

  src/declarativeimports/core/colorscope.h 38e7cc8 
  src/declarativeimports/core/colorscope.cpp 98ff3ab 
  src/declarativeimports/core/quicktheme.h 79db53c 
  src/declarativeimports/core/quicktheme.cpp 3ef5b88 
  src/declarativeimports/plasmastyle/TextAreaStyle.qml f5be27c 
  src/declarativeimports/plasmastyle/TextFieldStyle.qml 447d7e3 
  src/plasma/private/theme_p.cpp 603cf0a 
  src/plasma/theme.h 3f49719 

Diff: https://git.reviewboard.kde.org/r/127637/diff/


Testing
-------


Thanks,

Marco Martin

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160412/5be1c332/attachment.html>


More information about the Plasma-devel mailing list