Review Request 122706: A KCModule base for QML based KCMs

[David Edmundson]

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



src/kcmoduleqml.h
<https://git.reviewboard.kde.org/r/122706/#comment52714>

    copyrights are so last year.



src/kcmoduleqml.h
<https://git.reviewboard.kde.org/r/122706/#comment52712>

    documentation. Especially the package loading.



src/kcmoduleqml.h
<https://git.reviewboard.kde.org/r/122706/#comment52713>

    When we do make a QML only system settings we're not going to want to use KCMModule at all, even if it's not shown?
    
    I imagine we'll end up writing a small QObject shim that copies the API.
    
    I'd just leave this until we find we need it, otherwise we're committing to a slightly weird API prematurely.



src/kcmoduleqml.cpp
<https://git.reviewboard.kde.org/r/122706/#comment52701>

    leak?


- David Edmundson


On Feb. 24, 2015, 10:39 a.m., Marco Martin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/122706/
> -----------------------------------------------------------
> 
> (Updated Feb. 24, 2015, 10:39 a.m.)
> 
> 
> Review request for KDE Frameworks.
> 
> 
> Repository: kconfigwidgets
> 
> 
> Description
> -------
> 
> this adds a new class called KCModuleQml
> it loads a KPackage with the same plugin name as the kcm, then loads its mainscript qml file from it and puts it in a QQuickView used as main widget of the KCM.
> 
> It supports two ways of loading the qml:
> one is in the showevent of the KCM qwidget, this is compatible with current Systemsettings and kcmshell.
> the other way is with the mainUi accessor/qproperty. This will make possible for a pure QML version of systemsettings (accessing directly mainUi without showing qwidgets)
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt f3aaf18 
>   src/CMakeLists.txt 10862c6 
>   src/kcmoduleqml.h PRE-CREATION 
>   src/kcmoduleqml.cpp PRE-CREATION 
> 
> Diff: https://git.reviewboard.kde.org/r/122706/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Marco Martin
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20150224/23ba2606/attachment-0001.html>