Review Request: Load and save contour-polygon, and wrapping around it in Words

Thorsten Zachmann t.zachmann at zagge.de
Sun May 13 10:37:54 BST 2012


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

Ship it!


After fixing the two things I pointed out it is good to get in.


libs/flake/KoPathShape.cpp
<http://git.reviewboard.kde.org/r/104873/#comment10933>

    I think that should also check for activeControlPoint2



libs/textlayout/KoTextLayoutObstruction.cpp
<http://git.reviewboard.kde.org/r/104873/#comment10932>

    Please move inside the if as ithe clipPath is not needed in the else case


- Thorsten Zachmann


On May 13, 2012, 9:19 a.m., C. Boemann wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/104873/
> -----------------------------------------------------------
> 
> (Updated May 13, 2012, 9:19 a.m.)
> 
> 
> Review request for Calligra.
> 
> 
> Description
> -------
> 
> Load and save the contour-polygon and contour-path elements.
> They are stored as KoClipPath in the engine. Previously the engine only
> had support for loading and saving KoClipPath in svg
> 
> Load and save the style frame style attributes that turn contoured text run around
> (wrapping) on and off. wrap-contour and wrap-contour-mode
> 
> Use this new technology to do tight run around (simple change only requesting what we already had)
> 
> This commit also adds a method to pictureshape to create a tightfitting outline path of the image.
> It's tested and works, but is not used yet to create a KoClipPath out of it.
> 
> 
> Diffs
> -----
> 
>   libs/flake/KoPathShape.h d6e7cf1 
>   libs/flake/KoPathShape.cpp fc59f87 
>   libs/flake/KoShape.h 3b2ee21 
>   libs/flake/KoShape.cpp 40c2be2 
>   libs/flake/KoShape_p.h a8ef367 
>   libs/textlayout/KoTextLayoutObstruction.cpp 9711578 
>   plugins/pictureshape/PictureShape.h 54cc813 
>   plugins/pictureshape/PictureShape.cpp e4f58fe 
> 
> Diff: http://git.reviewboard.kde.org/r/104873/diff/
> 
> 
> Testing
> -------
> 
> I've created such contour clipping in LO
> 
> loaded and seen it work in Words, and saved it back to LO where it works (though we need to work around a bug in LO, workaround is there but not enabled in this diff)
> 
> For the unused method in the pictre shape I had hardwired it during development. I will put up a new review when I actually use the method so you may ignore it for now.
> 
> 
> Thanks,
> 
> C. Boemann
> 
>

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


More information about the calligra-devel mailing list