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

C. Boemann cbr at boemann.dk
Sun May 13 10:19:05 BST 2012


-----------------------------------------------------------
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.


Changes
-------

forgot to update the saving of viewbox


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 (updated)
-----

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


More information about the calligra-devel mailing list