Review Request 119527: Added feature of creating report's items of specific size at adding time

Jarosław Staniek staniek at kde.org
Sat Aug 30 10:41:33 BST 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119527/#review65537
-----------------------------------------------------------


Much better! In addition to fixing details explained below, could you please update the patch against the current master? It does not apply. Thanks.


libs/koreport/items/field/KoReportDesignerItemField.cpp
<https://git.reviewboard.kde.org/r/119527/#comment45789>

    conding style: extra (  ) not needed



libs/koreport/items/label/KoReportDesignerItemLabel.cpp
<https://git.reviewboard.kde.org/r/119527/#comment45791>

    coding style, too many ( )



libs/koreport/wrtembed/KoReportDesigner.cpp
<https://git.reviewboard.kde.org/r/119527/#comment45792>

    coding style: {



libs/koreport/wrtembed/KoReportDesigner.cpp
<https://git.reviewboard.kde.org/r/119527/#comment45793>

    coding style: {



libs/koreport/wrtembed/KoReportDesigner.cpp
<https://git.reviewboard.kde.org/r/119527/#comment45794>

    coding style: {



libs/koreport/wrtembed/KoReportDesigner.cpp
<https://git.reviewboard.kde.org/r/119527/#comment45795>

    coding style: {



libs/koreport/items/check/KoReportDesignerItemCheck.h
<https://git.reviewboard.kde.org/r/119527/#comment45788>

    *too many ';'
    *please use parameter names



libs/koreport/items/check/KoReportDesignerItemCheck.cpp
<https://git.reviewboard.kde.org/r/119527/#comment45797>

    Could const KoReportDesigner & be enough?



libs/koreport/items/image/KoReportDesignerItemImage.h
<https://git.reviewboard.kde.org/r/119527/#comment45790>

    too many ;



plugins/reporting/maps/KoReportDesignerItemMaps.h
<https://git.reviewboard.kde.org/r/119527/#comment45796>

    too many ;


- Jarosław Staniek


On Aug. 10, 2014, 10:28 p.m., Wojciech Kosowicz wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/119527/
> -----------------------------------------------------------
> 
> (Updated Aug. 10, 2014, 10:28 p.m.)
> 
> 
> Review request for Calligra, Adam Pigg and Jarosław Staniek.
> 
> 
> Bugs: 334967
>     http://bugs.kde.org/show_bug.cgi?id=334967
> 
> 
> Repository: calligra
> 
> 
> Description
> -------
> 
> If the specified dimensions are improper or too small elements get their defaults size
> 
> 
> Diffs
> -----
> 
>   libs/koreport/items/check/KoReportDesignerItemCheck.h 2ef6759 
>   libs/koreport/items/check/KoReportDesignerItemCheck.cpp 5bebc6c 
>   libs/koreport/items/field/KoReportDesignerItemField.h fef7346 
>   libs/koreport/items/field/KoReportDesignerItemField.cpp 0dc9325 
>   libs/koreport/items/image/KoReportDesignerItemImage.h c7f3484 
>   libs/koreport/items/image/KoReportDesignerItemImage.cpp 87e46a1 
>   libs/koreport/items/label/KoReportDesignerItemLabel.h b2aa78f 
>   libs/koreport/items/label/KoReportDesignerItemLabel.cpp 8974a80 
>   libs/koreport/items/text/KoReportDesignerItemText.h 19ba551 
>   libs/koreport/items/text/KoReportDesignerItemText.cpp 51e12d3 
>   libs/koreport/wrtembed/KoReportDesigner.h adc712a 
>   libs/koreport/wrtembed/KoReportDesigner.cpp 63152a7 
>   libs/koreport/wrtembed/KoReportDesignerItemBase.h de94858 
>   libs/koreport/wrtembed/KoReportDesignerItemBase.cpp 231771e 
>   libs/koreport/wrtembed/KoReportDesignerItemLine.h 894cec9 
>   libs/koreport/wrtembed/KoReportDesignerItemLine.cpp 1b0474d 
>   libs/koreport/wrtembed/KoReportDesignerItemRectBase.h b53a58e 
>   libs/koreport/wrtembed/KoReportDesignerItemRectBase.cpp 53c3727 
>   libs/koreport/wrtembed/reportsceneview.h cc3ab9f 
>   libs/koreport/wrtembed/reportsceneview.cpp 2b679a4 
>   plugins/reporting/barcode/KoReportDesignerItemBarcode.h 384863b 
>   plugins/reporting/barcode/KoReportDesignerItemBarcode.cpp b4fb621 
>   plugins/reporting/maps/KoReportDesignerItemMaps.h 56fbb6b 
>   plugins/reporting/maps/KoReportDesignerItemMaps.cpp f624db3 
>   plugins/reporting/web/KoReportDesignerItemWeb.h d2d7ce0 
>   plugins/reporting/web/KoReportDesignerItemWeb.cpp 63e1cc9 
> 
> Diff: https://git.reviewboard.kde.org/r/119527/diff/
> 
> 
> Testing
> -------
> 
> When adding any element in kexi report it will the same size as specified by the user at the moment of adding
> 
> 
> Thanks,
> 
> Wojciech Kosowicz
> 
>

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


More information about the calligra-devel mailing list