On Feb. 23, 2015, 11:08 a.m., Marco Martin wrote:
> > I like where this is going! :)
> 
> Marco Martin wrote:
>     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
> 
> Aleix Pol Gonzalez wrote:
>     I agree that being able to define the KAboutData would be interesting 
> eventually. It's probably out of scope for this patch.

well, it already creates a KAboutData, just that it doesn't use it..
since commandline parameters (even tough in a dynamic fashion could be done 
only in a very primitive fashion) would be needed from the get go, so a nice 
expandable interface wit hthe qml part would probably be nice


- 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
> 
>

_______________________________________________
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Reply via email to