Review Request: Anchored shape positioning fix

[Sebastian Sauer]

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

Ship it!


More testing done and it indeed looks correct and I didn't found any regressions. So "Shit It" :)


- Sebastian


On June 7, 2011, 12:49 p.m., Matus Hanzes wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/101532/
> -----------------------------------------------------------
> 
> (Updated June 7, 2011, 12:49 p.m.)
> 
> 
> Review request for Calligra, Casper Boemann and Sebastian Sauer.
> 
> 
> Summary
> -------
> 
> This is reimplementation of the old layout anchor logic.
> 
> The idea behind it is to releayout the text after every successful positioning of wrapping shape.
> And let it there where it was positioned the first time.
> 
> 
> Diffs
> -----
> 
>   libs/textlayout/FloatingAnchorStrategy.cpp f5b780c 
>   libs/textlayout/InlineAnchorStrategy.cpp 24908ee 
>   libs/textlayout/KoTextDocumentLayout.cpp 3819b22 
>   libs/textlayout/KoTextLayoutArea.cpp c822879 
> 
> Diff: http://git.reviewboard.kde.org/r/101532/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Matus
> 
>

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