----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/111044/ -----------------------------------------------------------
(Updated July 19, 2013, 6:26 a.m.) Review request for Calligra, C. Boemann, Boudewijn Rempt, and Thorsten Zachmann. Changes ------- This is the next update for the patch. This time it includes patches from boemann too, that implement handling of padding inside the borders. This makes the test file look even better. Description ------- This patch implements loading, saving and painting of borders for shapes in general. It also implements calls to actually paint borders in particular for the picture shape and text shape. At the same time I moved border painting from KWCanvasBase to KoBorder, which I announced some weeks ago that I wanted to do. A patch this deep in the libraries should not go in without a run of cstester. I would be glad for help with this since I only have a semi-weak laptop. Diffs (updated) ----- libs/flake/KoShape.h fe6cc3f libs/flake/KoShape.cpp 6d14a8c libs/flake/KoShape_p.h aa6633c libs/odf/KoBorder.h bdbf86e libs/odf/KoBorder.cpp 58f8e45 libs/textlayout/FloatingAnchorStrategy.h eb0ff6b libs/textlayout/FloatingAnchorStrategy.cpp 465f149 libs/textlayout/KoTextDocumentLayout.cpp 70f4ccd libs/textlayout/KoTextLayoutRootAreaProvider.h d8c7ac3 libs/textlayout/KoTextShapeData.cpp 1b424b7 plugins/pictureshape/PictureShape.cpp 976fadd plugins/textshape/SimpleRootAreaProvider.h 62534a6 plugins/textshape/SimpleRootAreaProvider.cpp 63f1898 plugins/textshape/TextShape.cpp b230f0a words/part/KWCanvasBase.cpp 7d12890 words/part/KWRootAreaProvider.h 45b4870 words/part/KWRootAreaProvider.cpp fc8660d Diff: http://git.reviewboard.kde.org/r/111044/diff/ Testing ------- Tested with all the files with the string "border" in it in the test suite. Thanks, Inge Wallin
_______________________________________________ calligra-devel mailing list calligra-devel@kde.org https://mail.kde.org/mailman/listinfo/calligra-devel