Review Request: Anchored shape positioning fix

[Casper Boemann]

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


Thanks for working on this. I'm no totally against it, but just a bit worried it's a gainst the odf spec, so I'm going to have to take a closer look.



 I think it's good, but as i said i really neeed to look this through which is probably going to take more than a day, considering i'm on vacation. Sebastian you can look too please, but i'd like to at least have a say before we okay this 

- Casper


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/a3cffa2b/attachment.htm>