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

(Updated May 5, 2016, 9:54 a.m.)


Review request for KDE Frameworks and Plasma.


Changes
-------

remove buttonhighlightedtext,viewhighlightedtext etc


as there is only one highlight color, there is also only one highlighted text 
color.

those entries were added after last frameworks release, so can still be removed


Repository: plasma-framework


Description
-------

lately the highlight color of plasma became a very washed-out blue, (since it 
switched to use the global system colors) that was because it actually taken 
the highlight color (and highlightedText) from the wrong source.
there is a correct color group in kcolorscheme for that, that is Selection, 
take those two colors from that (also means highlight and highlightedtext 
always be the same in all plasma::theme::colorgroups but that's not a problem)

update test to reflect that


Diffs (updated)
-----

  autotests/themetest.cpp ce512a4 
  src/declarativeimports/core/quicktheme.h ac5e121 
  src/plasma/private/theme_p.h d2246de 
  src/plasma/private/theme_p.cpp e6d55c3 

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


Testing
-------


Thanks,

Marco Martin

_______________________________________________
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Reply via email to