Review Request: Changed KoReport to use QGraphicsView instead of QWidget

Jarosław Staniek staniek at kde.org
Sat May 19 23:31:04 BST 2012


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/104998/#review13989
-----------------------------------------------------------



kexi/plugins/reports/kexireportview.cpp
<http://git.reviewboard.kde.org/r/104998/#comment11094>

    could better use palette().brush(QPalette::Dark) as QMDI does



kexi/plugins/reports/kexireportview.cpp
<http://git.reviewboard.kde.org/r/104998/#comment11096>

    BTW, Table -> HTML Table


- Jarosław Staniek


On May 19, 2012, 9:59 p.m., Adam Pigg wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/104998/
> -----------------------------------------------------------
> 
> (Updated May 19, 2012, 9:59 p.m.)
> 
> 
> Review request for Calligra, Jarosław Staniek and Dag Andersen.
> 
> 
> Description
> -------
> 
> Changed KoReport to use QGraphicsView instead of QWidget
> 
> Use QGraphicsView and QGraphicsItem for the report page view
> instead of painting a custom QWidget.  This will make it easier
> to use in QML, and also happens to look nicer.
> 
> 
> Diffs
> -----
> 
>   kexi/plugins/reports/kexireportview.h 952e0dae57ac2c4f86bfcc9da40dbb6ffeda0746 
>   kexi/plugins/reports/kexireportview.cpp ccdf3f91453ca26a00ae8ac3f0976cac7c04c72f 
>   libs/koreport/renderer/KoReportPage.h 65bf612d4bb02320a6e757b32fb2ce1e254f44fd 
>   libs/koreport/renderer/KoReportPage.cpp e0fafb6a10c07b31bdb03e08bf162f85053ea2ad 
> 
> Diff: http://git.reviewboard.kde.org/r/104998/diff/
> 
> 
> Testing
> -------
> 
> Tested in Kexi using existing reports, but breaks the build of Plan as it will need a new report view widget based on QGraphicsView.  I will add this later, the same as the Kexi version.
> 
> 
> Thanks,
> 
> Adam Pigg
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20120519/d62a7435/attachment.htm>


More information about the calligra-devel mailing list