Review Request: The basic web widget in Kexi reports, put up for review.The widget is not yet painted on report canvas.Please suggest possible issues

Adam Pigg adam at piggz.co.uk
Fri Jul 22 12:15:29 CEST 2011


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



plugins/reporting/web/KoReportItemweb.cpp
<http://git.reviewboard.kde.org/r/102046/#comment4360>

    To render as an item on a page, you want to do something like
    OROPicture * pic = new OROPicture();
            widget()->setFixedSize(m_size.toScene().toSize());
    
            QPainter p(pic->picture());
    
            widget->paint(&p, QRect(QPoint(0, 0), m_size.toScene().toSize()));
    
            QPointF pos = m_pos.toScene();
            QSizeF size = m_size.toScene();
    
            pos += offset;
    
            pic->setPosition(pos);
            pic->setSize(size);
            if (page) page->addPrimitive(pic);
    
    


- Adam


On July 22, 2011, 9:22 a.m., Shreya Pandit wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/102046/
> -----------------------------------------------------------
> 
> (Updated July 22, 2011, 9:22 a.m.)
> 
> 
> Review request for Calligra and Jarosław Staniek.
> 
> 
> Summary
> -------
> 
> Web widget for reports.Please suggest changes
> 
> 
> This addresses bug no.
>     http://bugs.kde.org/show_bug.cgi?id=no
> 
> 
> Diffs
> -----
> 
>   plugins/reporting/CMakeLists.txt 42960d6 
>   plugins/reporting/web/KoReportDesignerItemweb.h PRE-CREATION 
>   plugins/reporting/web/KoReportDesignerItemweb.cpp 6c1b43b 
>   plugins/reporting/web/KoReportItemweb.h 07849c3 
>   plugins/reporting/web/KoReportItemweb.cpp cc5c7b9 
>   plugins/reporting/web/KoReportwebPlugin.h PRE-CREATION 
>   plugins/reporting/web/KoReportwebPlugin.cpp PRE-CREATION 
> 
> Diff: http://git.reviewboard.kde.org/r/102046/diff
> 
> 
> Testing
> -------
> 
> built
> 
> 
> Thanks,
> 
> Shreya
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/calligra-devel/attachments/20110722/abc46a83/attachment.htm 


More information about the calligra-devel mailing list