Review Request: Port the Calculator applet to QML
Aaron J. Seigo
aseigo at kde.org
Thu Nov 8 09:48:53 UTC 2012
> On Nov. 8, 2012, 9:33 a.m., Marco Martin wrote:
> > style is quite ok to me now.
> > functions could still be changed to
> >
> > function foo()
> > {
> > ...
> > }
actually, the style was finally updated to:
function foo() {
}
so no need to adjust this :)
- Aaron J.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/107001/#review21614
-----------------------------------------------------------
On Nov. 6, 2012, 11:36 p.m., Romário Rios wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/107001/
> -----------------------------------------------------------
>
> (Updated Nov. 6, 2012, 11:36 p.m.)
>
>
> Review request for Plasma.
>
>
> Description
> -------
>
> This diff replaces the C++ Calculator applet by its QMLfied version, which seems to be feature-par with the latter.
>
>
> Diffs
> -----
>
> CMakeLists.txt 732145c
> calculator.h f7339be
> calculator.cpp 6bc3ddc
> package/contents/ui/calculator.qml PRE-CREATION
> package/metadata.desktop PRE-CREATION
> plasma-applet-calculator.desktop 0760729
>
> Diff: http://git.reviewboard.kde.org/r/107001/diff/
>
>
> Testing
> -------
>
> Tested as both applet and popupapplet, with the Air theme and the Oxygen theme. It seems to work fine.
>
>
> Thanks,
>
> Romário Rios
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20121108/b3e21d9c/attachment.html>
More information about the Plasma-devel
mailing list