--===============1505135141== Content-Type: multipart/alternative; boundary="===============7855759692471061075==" --===============7855759692471061075== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable > On 2010-10-22 11:56:30, Sven Langkamp wrote: > > I noticed that the patch still uses KisShapeLayer::addChild that should= not with the ShapeLayerContainerModel Yeah, they are identical atm. I wanted to deprecate addChild in the next pa= tch. Btw, there is a problem with it: addChild has an optional parameter th= at disables updates and it is used in a single place. I haven't got an idea= how to workaround it yet. Do you have one? - Dmitry ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://svn.reviewboard.kde.org/r/5680/#review8303 ----------------------------------------------------------- On 2010-10-21 20:58:38, Dmitry Kazakov wrote: > = > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://svn.reviewboard.kde.org/r/5680/ > ----------------------------------------------------------- > = > (Updated 2010-10-21 20:58:38) > = > = > Review request for KOffice. > = > = > Summary > ------- > = > This patch fixes many bugs in shape layers. It consists of two small part= s: the first makes KisBaseNode::visible() recursive, the second brings orde= r to KisShapeController. > = > Need help with testing. > = > = > This addresses bugs 252451, 253766 and 254178. > https://bugs.kde.org/show_bug.cgi?id=3D252451 > https://bugs.kde.org/show_bug.cgi?id=3D253766 > https://bugs.kde.org/show_bug.cgi?id=3D254178 > = > = > Diffs > ----- > = > /trunk/koffice/krita/image/kis_base_node.h 1183235 = > /trunk/koffice/krita/image/kis_base_node.cpp 1183235 = > /trunk/koffice/krita/image/kis_node.h 1183235 = > /trunk/koffice/krita/image/kis_node.cpp 1183235 = > /trunk/koffice/krita/ui/CMakeLists.txt 1183235 = > /trunk/koffice/krita/ui/canvas/kis_canvas2.cpp 1183235 = > /trunk/koffice/krita/ui/flake/kis_node_shape.cpp 1183235 = > /trunk/koffice/krita/ui/flake/kis_shape_controller.h 1183235 = > /trunk/koffice/krita/ui/flake/kis_shape_controller.cpp 1183235 = > /trunk/koffice/krita/ui/flake/kis_shape_layer.h 1183235 = > /trunk/koffice/krita/ui/flake/kis_shape_layer.cc 1183235 = > /trunk/koffice/krita/ui/flake/kis_shape_selection.h 1183235 = > /trunk/koffice/krita/ui/tests/kis_shape_controller_test.cpp 1183235 = > /trunk/koffice/krita/ui/tool/kis_selection_tool_helper.cpp 1183235 = > = > Diff: http://svn.reviewboard.kde.org/r/5680/diff > = > = > Testing > ------- > = > = > Thanks, > = > Dmitry > = > --===============7855759692471061075== Content-Type: text/html; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable
This is an automatically generated e-mail. To reply, visit: http://svn.reviewb= oard.kde.org/r/5680/ |
On October 22nd, 2010, 11:56 a.m., Sven Lan= gkamp wrote:
I noticed= that the patch still uses KisShapeLayer::addChild that should not with the= ShapeLayerContainerModel
Yeah, they = are identical atm. I wanted to deprecate addChild in the next patch. Btw, t= here is a problem with it: addChild has an optional parameter that disables= updates and it is used in a single place. I haven't got an idea how to= workaround it yet. Do you have one?
- Dmitry
On October 21st, 2010, 8:58 p.m., Dmitry Kazakov wrote:
Review request for KOffice.
By Dmitry Kazakov.
Updated 2010-10-21 20:58:38 Descripti= on
Diffs=
|