Review Request 126654: rework ColorScope, hopefully fix black on black battery on phone
Marco Martin
notmart at gmail.com
Thu Jan 7 10:07:01 UTC 2016
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/126654/
-----------------------------------------------------------
(Updated Jan. 7, 2016, 10:07 a.m.)
Status
------
This change has been marked as submitted.
Review request for Plasma and Bhushan Shah.
Changes
-------
Submitted with commit e565c5c225a1eb7721647f36bf28280e6d8ee0a2 by Marco Martin to branch master.
Repository: plasma-framework
Description
-------
Since some time PlasmaCore had the very useful ColorScope class,
which can semi-automatically make children take colors of the
View, Button or Complementary color groups.
Unfortunately it didn't support dynamic reparenting, giving
wrong colors in some cases (see black on black battery on the phone)
even tough is a bit complicated, now by cascading signals down the
children tree, support on the fly apply of the new context upon
reparenting.
also add a manual test case showing text, rectangles and icons
dynamically changing color as they get moved from one scope
to the other
Diffs
-----
tests/colorcontext.qml PRE-CREATION
src/declarativeimports/core/colorscope.cpp 92dcb3e
src/declarativeimports/core/colorscope.h 93b3303
Diff: https://git.reviewboard.kde.org/r/126654/diff/
Testing
-------
File Attachments
----------------
context1.png
https://git.reviewboard.kde.org/media/uploaded/files/2016/01/06/0f1bf140-9ac3-4ca8-9f42-c6d51c1dad3c__context1.png
context2.png
https://git.reviewboard.kde.org/media/uploaded/files/2016/01/06/a65435ce-bec6-4272-b353-5c5d4a4178fb__context2.png
Thanks,
Marco Martin
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160107/12f9dc05/attachment.html>
More information about the Plasma-devel
mailing list