Review Request 125951: [calendar] Move the plugins handling to a separate class
Martin Klapetek
martin.klapetek at gmail.com
Mon Nov 9 19:40:34 UTC 2015
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/125951/
-----------------------------------------------------------
(Updated Nov. 9, 2015, 8:40 p.m.)
Review request for Plasma.
Changes
-------
Redo the config model a bit
I'm still not sure if I can parent to the engine or not..?
Repository: plasma-framework
Description
-------
This is also made a QML singleton that will be used for the applet
config view where it will add the plugin configs once we add that
possibility.
The same instance is then set to the DaysModel from QML.
(this depends on https://git.reviewboard.kde.org/r/125817/ which awaits ship it)
Diffs (updated)
-----
src/declarativeimports/calendar/CMakeLists.txt 40ead911ad5208cae5dbe5333d227f9f8a0d9154
src/declarativeimports/calendar/calendarplugin.cpp bafe80cf7520a08312abfd1dbd6d4648a6710175
src/declarativeimports/calendar/daysmodel.h a5bdac98627f7efa76bd4afd239469b53e06690b
src/declarativeimports/calendar/daysmodel.cpp 2d059a8e8636565adbe52811e602fff37a5eb157
src/declarativeimports/calendar/eventpluginsmanager.h PRE-CREATION
src/declarativeimports/calendar/eventpluginsmanager.cpp PRE-CREATION
src/declarativeimports/calendar/qml/MonthView.qml f698934f850ef3a917b9611c9f9a40c369b23f6c
Diff: https://git.reviewboard.kde.org/r/125951/diff/
Testing
-------
Calendar events are still correctly displayed
Thanks,
Martin Klapetek
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20151109/318b4666/attachment.html>
More information about the Plasma-devel
mailing list