Review Request 122988: make PlasmaCore.ColorScope work better as advertised
Marco Martin
notmart at gmail.com
Wed Mar 18 17:43:18 UTC 2015
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/122988/
-----------------------------------------------------------
(Updated March 18, 2015, 5:43 p.m.)
Review request for Plasma.
Repository: plasma-framework
Description
-------
Since attached properties cannot ever change during the lifetime of the object, a colorscope needs to be created for each obect that asks for the attached property.
now each instance if (internal) has m_inherit (only those created as an attached property), will search for the property of the proper ancestor.
Not super efficient, but the proeprties should be actually asked for rarely enough.
doesn't look much nice, but seems to work pretty reliably and the only way to.
Diffs (updated)
-----
src/declarativeimports/core/colorscope.h fa75b9d
src/declarativeimports/core/colorscope.cpp cca0d2f
src/declarativeimports/core/framesvgitem.h 20486c7
src/declarativeimports/core/framesvgitem.cpp e2a0789
Diff: https://git.reviewboard.kde.org/r/122988/diff/
Testing
-------
Thanks,
Marco Martin
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150318/23df49a3/attachment.html>
More information about the Plasma-devel
mailing list