Review Request: make it possible to read properties from different property types.
Jan Hambrecht
jaham at gmx.net
Sun Oct 2 12:03:01 BST 2011
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/102737/#review6997
-----------------------------------------------------------
Ship it!
I think beside the two things pointed out you could commit.
libs/odf/KoStyleStack.cpp
<http://git.reviewboard.kde.org/r/102737/#comment6177>
I think beside checking for a null pointer a check for an empty string would be good too.
libs/odf/KoStyleStack.cpp
<http://git.reviewboard.kde.org/r/102737/#comment6176>
I think it would be good to check for an empty string here.
- Jan Hambrecht
On Sept. 29, 2011, 1:28 p.m., Thorsten Zachmann wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/102737/
> -----------------------------------------------------------
>
> (Updated Sept. 29, 2011, 1:28 p.m.)
>
>
> Review request for Calligra.
>
>
> Description
> -------
>
> Make it possible to load style properties from multiple properties.
>
> For embedded text shapes paragraph style properties can be saved in style:paragraph-properties or in graphic:paragraph-properties.
> It is also possible that some values are in one and others are in the other e.g. in parent styles.
>
> This patch add the possibilty to load the properties from more then one property type.
>
> This partly fixes the bug in the bug report attached. It is not fully fixed as not all properties are used by the text shape yet.
>
>
> This addresses bug 282617.
> http://bugs.kde.org/show_bug.cgi?id=282617
>
>
> Diffs
> -----
>
> libs/kotext/styles/KoParagraphStyle.cpp 9ecd77a
> libs/odf/KoStyleStack.h a087dc3
> libs/odf/KoStyleStack.cpp d94025a
>
> Diff: http://git.reviewboard.kde.org/r/102737/diff/diff
>
>
> Testing
> -------
>
> Run with cstester and found some nice improvements.
>
>
> Thanks,
>
> Thorsten Zachmann
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20111002/760cb6ef/attachment.htm>
More information about the calligra-devel
mailing list