<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/102809/">http://git.reviewboard.kde.org/r/102809/</a>
</td>
</tr>
</table>
<br />
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">This review has been submitted with commit f3a8582f55a6cffcc37b378d7f37651ebd55073d by C. Boemann to branch master.</pre>
<br />
<p>- Commit</p>
<br />
<p>On October 9th, 2011, 5:10 p.m., C. Boemann wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Calligra.</div>
<div>By C. Boemann.</div>
<p style="color: grey;"><i>Updated Oct. 9, 2011, 5:10 p.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">This patch passes the canvas to the paint and paintComponent methods of KoShape and decendents.
The reason is that some shapes (notably the text shape) will like to query the canvasResourceManager for info about such things as if it should paint:
- borders around tables
- decoration of spellchecking
- formatting chars (line break, spaces, tabs)
The alternative is to pass some form of a paint context, but since we already pass the canvas to the paintDecorations method and it gives access to the resourcemanager I thought this might be as good a solution as any other.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>braindump/plugins/stateshape/StateShape.h <span style="color: grey">(b8f79d6)</span></li>
<li>braindump/plugins/stateshape/StateShape.cpp <span style="color: grey">(3fae42e)</span></li>
<li>braindump/plugins/webshape/WebShape.h <span style="color: grey">(bdc6583)</span></li>
<li>braindump/plugins/webshape/WebShape.cpp <span style="color: grey">(b4abf0b)</span></li>
<li>braindump/src/Layout.cpp <span style="color: grey">(2cad88a)</span></li>
<li>karbon/plugins/tools/CalligraphyTool/KarbonCalligraphyTool.cpp <span style="color: grey">(e137387)</span></li>
<li>karbon/plugins/tools/KarbonPencilTool.cpp <span style="color: grey">(5d9533d)</span></li>
<li>krita/ui/flake/kis_layer_container_shape.h <span style="color: grey">(ae603ff)</span></li>
<li>krita/ui/flake/kis_layer_container_shape.cc <span style="color: grey">(a9c50d3)</span></li>
<li>krita/ui/flake/kis_node_shape.h <span style="color: grey">(5567c7d)</span></li>
<li>krita/ui/flake/kis_node_shape.cpp <span style="color: grey">(6e695a6)</span></li>
<li>krita/ui/flake/kis_shape_selection.h <span style="color: grey">(c932aa3)</span></li>
<li>krita/ui/flake/kis_shape_selection.cpp <span style="color: grey">(0b72764)</span></li>
<li>libs/flake/CMakeLists.txt <span style="color: grey">(1d34c3e)</span></li>
<li>libs/flake/KoCanvasControllerWidgetViewport_p.cpp <span style="color: grey">(345f51a)</span></li>
<li>libs/flake/KoPathShape.h <span style="color: grey">(8b54f26)</span></li>
<li>libs/flake/KoPathShape.cpp <span style="color: grey">(4c5ffb6)</span></li>
<li>libs/flake/KoSelection.h <span style="color: grey">(a8d14bf)</span></li>
<li>libs/flake/KoSelection.cpp <span style="color: grey">(0c4f7ed)</span></li>
<li>libs/flake/KoShape.h <span style="color: grey">(6ca2ede)</span></li>
<li>libs/flake/KoShape.cpp <span style="color: grey">(6b07e7c)</span></li>
<li>libs/flake/KoShapeContainer.h <span style="color: grey">(483d893)</span></li>
<li>libs/flake/KoShapeContainer.cpp <span style="color: grey">(d9b466a)</span></li>
<li>libs/flake/KoShapeGroup.h <span style="color: grey">(12ca7b4)</span></li>
<li>libs/flake/KoShapeGroup.cpp <span style="color: grey">(0515dce)</span></li>
<li>libs/flake/KoShapeLayer.h <span style="color: grey">(c098b9a)</span></li>
<li>libs/flake/KoShapeLayer.cpp <span style="color: grey">(a17f44b)</span></li>
<li>libs/flake/KoShapeManager.h <span style="color: grey">(9baa4b3)</span></li>
<li>libs/flake/KoShapeManager.cpp <span style="color: grey">(c3b3484)</span></li>
<li>libs/flake/KoShapeManagerPaintingStrategy.h <span style="color: grey">(5a62b23)</span></li>
<li>libs/flake/KoShapeManagerPaintingStrategy.cpp <span style="color: grey">(4a267c5)</span></li>
<li>libs/flake/KoShapePaintingContext.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>libs/flake/KoShapePaintingContext.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>libs/flake/KoTosContainer.h <span style="color: grey">(98c0f55)</span></li>
<li>libs/flake/KoTosContainer.cpp <span style="color: grey">(3a0aa51)</span></li>
<li>libs/flake/KoUnavailShape.h <span style="color: grey">(e8f1cf4)</span></li>
<li>libs/flake/KoUnavailShape.cpp <span style="color: grey">(e06a8b0)</span></li>
<li>libs/flake/tests/MockShapes.h <span style="color: grey">(5de9cd2)</span></li>
<li>libs/flake/tests/TestShapePainting.cpp <span style="color: grey">(871fe1c)</span></li>
<li>libs/flake/tools/KoCreatePathTool.cpp <span style="color: grey">(fb4a4ea)</span></li>
<li>libs/kopageapp/KoPAPageBase.h <span style="color: grey">(de22a80)</span></li>
<li>libs/kopageapp/KoPAPageBase.cpp <span style="color: grey">(09e2ae3)</span></li>
<li>plugins/artistictextshape/ArtisticTextShape.h <span style="color: grey">(04cc365)</span></li>
<li>plugins/artistictextshape/ArtisticTextShape.cpp <span style="color: grey">(a7835e3)</span></li>
<li>plugins/chartshape/ChartShape.h <span style="color: grey">(a65d6fa)</span></li>
<li>plugins/chartshape/ChartShape.cpp <span style="color: grey">(34db6ee)</span></li>
<li>plugins/chartshape/Legend.h <span style="color: grey">(ac4a419)</span></li>
<li>plugins/chartshape/Legend.cpp <span style="color: grey">(141ecf0)</span></li>
<li>plugins/chartshape/PlotArea.h <span style="color: grey">(85ce07b)</span></li>
<li>plugins/chartshape/PlotArea.cpp <span style="color: grey">(63e425b)</span></li>
<li>plugins/chartshape/TextLabelDummy.h <span style="color: grey">(85bfed2)</span></li>
<li>plugins/chartshape/TextLabelDummy.cpp <span style="color: grey">(4ca0792)</span></li>
<li>plugins/commentshape/CommentShape.h <span style="color: grey">(5b8e231)</span></li>
<li>plugins/commentshape/CommentShape.cpp <span style="color: grey">(0fd7d30)</span></li>
<li>plugins/commentshape/InitialsCommentShape.h <span style="color: grey">(1c49b72)</span></li>
<li>plugins/commentshape/InitialsCommentShape.cpp <span style="color: grey">(9c04211)</span></li>
<li>plugins/dockers/shapecollection/ShapeCollectionDocker.cpp <span style="color: grey">(719d569)</span></li>
<li>plugins/formulashape/KoFormulaShape.h <span style="color: grey">(0afc761)</span></li>
<li>plugins/formulashape/KoFormulaShape.cpp <span style="color: grey">(2aef65d)</span></li>
<li>plugins/musicshape/MusicShape.h <span style="color: grey">(6374d36)</span></li>
<li>plugins/musicshape/MusicShape.cpp <span style="color: grey">(d410360)</span></li>
<li>plugins/pictureshape/PictureShape.h <span style="color: grey">(f8bbc3e)</span></li>
<li>plugins/pictureshape/PictureShape.cpp <span style="color: grey">(66b3c83)</span></li>
<li>plugins/pluginshape/PluginShape.h <span style="color: grey">(e5a50cb)</span></li>
<li>plugins/pluginshape/PluginShape.cpp <span style="color: grey">(5840931)</span></li>
<li>plugins/textshape/ShrinkToFitShapeContainer.h <span style="color: grey">(39d2117)</span></li>
<li>plugins/textshape/ShrinkToFitShapeContainer.cpp <span style="color: grey">(7253479)</span></li>
<li>plugins/textshape/TextShape.h <span style="color: grey">(4ae4941)</span></li>
<li>plugins/textshape/TextShape.cpp <span style="color: grey">(5ee4bba)</span></li>
<li>plugins/textshape/dialogs/SimpleParagraphWidget.cpp <span style="color: grey">(5dbb238)</span></li>
<li>plugins/textshape/dialogs/TableOfContentsPreview.cpp <span style="color: grey">(272a7f7)</span></li>
<li>plugins/treeshape/TreeShape.h <span style="color: grey">(6370621)</span></li>
<li>plugins/treeshape/TreeShape.cpp <span style="color: grey">(756e2d6)</span></li>
<li>plugins/treeshape/TreeShapeMoveStrategy.cpp <span style="color: grey">(4184f32)</span></li>
<li>plugins/vectorshape/VectorShape.h <span style="color: grey">(cb5ab8e)</span></li>
<li>plugins/vectorshape/VectorShape.cpp <span style="color: grey">(c14e6a7)</span></li>
<li>plugins/videoshape/VideoShape.h <span style="color: grey">(d6bd829)</span></li>
<li>plugins/videoshape/VideoShape.cpp <span style="color: grey">(08e653e)</span></li>
<li>stage/part/KPrNotes.cpp <span style="color: grey">(598a1a0)</span></li>
<li>stage/part/KPrPlaceholderShape.h <span style="color: grey">(184a361)</span></li>
<li>stage/part/KPrPlaceholderShape.cpp <span style="color: grey">(03a9eba)</span></li>
<li>stage/part/KPrPlaceholderStrategy.h <span style="color: grey">(873eaa3)</span></li>
<li>stage/part/KPrPlaceholderStrategy.cpp <span style="color: grey">(6d0761f)</span></li>
<li>stage/part/KPrPlaceholderTextStrategy.h <span style="color: grey">(4202bc6)</span></li>
<li>stage/part/KPrPlaceholderTextStrategy.cpp <span style="color: grey">(9e99f9f)</span></li>
<li>stage/part/KPrShapeManagerAnimationStrategy.h <span style="color: grey">(052cbc6)</span></li>
<li>stage/part/KPrShapeManagerAnimationStrategy.cpp <span style="color: grey">(36fc516)</span></li>
<li>stage/part/KPrShapeManagerDisplayMasterStrategy.h <span style="color: grey">(1db5a7f)</span></li>
<li>stage/part/KPrShapeManagerDisplayMasterStrategy.cpp <span style="color: grey">(b6f2e26)</span></li>
<li>tables/shape/TableShape.h <span style="color: grey">(03b7e9b)</span></li>
<li>tables/shape/TableShape.cpp <span style="color: grey">(d53b069)</span></li>
<li>tables/ui/RightToLeftPaintingStrategy.h <span style="color: grey">(6faac6d)</span></li>
<li>tables/ui/RightToLeftPaintingStrategy.cpp <span style="color: grey">(9dede5c)</span></li>
<li>words/part/commands/KWClipFrameCommand.cpp <span style="color: grey">(b55aafe)</span></li>
<li>words/part/frames/KWCopyShape.h <span style="color: grey">(1ad91ac)</span></li>
<li>words/part/frames/KWCopyShape.cpp <span style="color: grey">(5a77481)</span></li>
<li>words/part/frames/KWFrameLayout.cpp <span style="color: grey">(9790f73)</span></li>
<li>words/part/frames/KWOutlineShape.h <span style="color: grey">(300bfef)</span></li>
<li>words/part/frames/KWOutlineShape.cpp <span style="color: grey">(4d04cfe)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/102809/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>