Review Request 122667: Some tools for easily do qml based apps using KPackage

[Marco Martin]

> On Feb. 23, 2015, 11:08 a.m., Aleix Pol Gonzalez wrote:
> > src/qmlpackagelauncher/CMakeLists.txt, line 8
> > <https://git.reviewboard.kde.org/r/122667/diff/1/?file=350611#file350611line8>
> >
> >     Maybe it would make sense to call it "kpackagelauncher-qml"?

renamed, but without the dash that i don't like too much


On Feb. 23, 2015, 11:08 a.m., Marco Martin wrote:
> > I like where this is going! :)

One thing i'm still not sure is what to export to it. it now sets the application commandline arguments, that's not really possible to obtain otherwise.
Another thing would probably be the kaboutdata, which now is constructing one but not really doing anything with it.
I would like to have an easy way to automatically have the various about application/about kde dialogs there too, even tough not depending from kxmlgui, but don't really know a clean way to do it atm


- Marco


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


On Feb. 25, 2015, 9:29 a.m., Marco Martin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/122667/
> -----------------------------------------------------------
> 
> (Updated Feb. 25, 2015, 9:29 a.m.)
> 
> 
> Review request for KDE Frameworks and Plasma.
> 
> 
> Repository: kdeclarative
> 
> 
> Description
> -------
> 
> This is still a bit rough but adding it there for early feedback. comes from the fact that I would like a standard-ish way for KDE apps to do QML based apps
> 
> It adds 3 things:
> * KPackage support to QmlObject: the class is done to easily load QML without using qquickview, since I want to push loading qml from KPackages, adding kpackage support to this class seems to make sense
> * qmlpackagelauncher: tiny tool to launch qml apps from qml packages: supports both the case the root object is a qquickitem or an ApplicationWindow qtquickcomponent(that is a window in itself) puts proper window title and icon, sets the kaboutdata from the metadata file, binds commandline arguments in the context (kinda feel some other bindings/API could be needed)
> * KPackageView: is a qquickview that loads kpackages... seeing that the direction from Qt of using ApplicationWindow as root item makes me a bit unsure wether to add this tough
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt 52a698a 
>   src/CMakeLists.txt 32da8a5 
>   src/kdeclarative/CMakeLists.txt 052b9e2 
>   src/kdeclarative/qmlobject.h a0a74d3 
>   src/kdeclarative/qmlobject.cpp 00478b4 
>   src/kpackagelauncherqml/CMakeLists.txt PRE-CREATION 
>   src/kpackagelauncherqml/main.cpp PRE-CREATION 
>   src/quickaddons/CMakeLists.txt 8272f1f 
>   tests/CMakeLists.txt a8abfaf 
>   tests/helloworld/contents/ui/main.qml PRE-CREATION 
>   tests/helloworld/metadata.desktop PRE-CREATION 
> 
> Diff: https://git.reviewboard.kde.org/r/122667/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Marco Martin
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150225/7bc0046b/attachment.html>