Review Request: Changing main.cpp to use KApplication in calligra active
Jarosław Staniek
staniek at kde.org
Wed Nov 7 19:50:26 GMT 2012
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/107244/#review21587
-----------------------------------------------------------
active/main.cpp
<http://git.reviewboard.kde.org/r/107244/#comment16769>
This is not a description aimed at target users but developers; I propose "Calligra application for mobile devices" to hide architectural details, otherwise packagers will pick up such description confusing for normal users.
- Jarosław Staniek
On Nov. 7, 2012, 6:11 p.m., Sujith Haridasan wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/107244/
> -----------------------------------------------------------
>
> (Updated Nov. 7, 2012, 6:11 p.m.)
>
>
> Review request for Calligra and Shantanu Tushar.
>
>
> Description
> -------
>
> Moving calligra from QApplication to KApplication. Made changes to main.cpp file. The reference was taken from plasma-mobile/shell.
>
>
> Diffs
> -----
>
> active/main.cpp 81a00c6
>
> Diff: http://git.reviewboard.kde.org/r/107244/diff/
>
>
> Testing
> -------
>
> Did some basic testing with presentation slide, Spread sheet and Word file. Also tried to navigate and search strings. It worked fine.
>
>
> Thanks,
>
> Sujith Haridasan
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20121107/e8e5b7cf/attachment.htm>
More information about the calligra-devel
mailing list