----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103152/#review8258 -----------------------------------------------------------
In general it looks like you have made the right choices as to where to do things. Looks nice ,though I wouldn't expect less from you :) I have not really reviewed the code as such. I did find a number of hacking style glitches though libs/flake/KoPathShape.cpp <http://git.reviewboard.kde.org/r/103152/#comment7049> spaces libs/flake/commands/KoPathShapeMarkerCommand.h <http://git.reviewboard.kde.org/r/103152/#comment7050> spaces libs/flake/commands/KoPathShapeMarkerCommand.h <http://git.reviewboard.kde.org/r/103152/#comment7051> spaces libs/widgets/KoMarkerSelector.cpp <http://git.reviewboard.kde.org/r/103152/#comment7052> { } libs/widgets/KoMarkerSelector.cpp <http://git.reviewboard.kde.org/r/103152/#comment7053> if <space> ( libs/widgets/KoStrokeConfigWidget.h <http://git.reviewboard.kde.org/r/103152/#comment7054> spaces plugins/dockers/strokedocker/StrokeDocker.h <http://git.reviewboard.kde.org/r/103152/#comment7055> spaces plugins/dockers/strokedocker/StrokeDocker.h <http://git.reviewboard.kde.org/r/103152/#comment7056> spaces plugins/dockers/strokedocker/StrokeDocker.cpp <http://git.reviewboard.kde.org/r/103152/#comment7058> ) <space> { plugins/dockers/strokedocker/StrokeDocker.cpp <http://git.reviewboard.kde.org/r/103152/#comment7057> spaces plugins/dockers/strokedocker/StrokeDocker.cpp <http://git.reviewboard.kde.org/r/103152/#comment7059> space and { } plugins/dockers/strokedocker/StrokeDocker.cpp <http://git.reviewboard.kde.org/r/103152/#comment7060> if <space> () <space> { - C. Boemann On Nov. 16, 2011, 5:51 a.m., Thorsten Zachmann wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/103152/ > ----------------------------------------------------------- > > (Updated Nov. 16, 2011, 5:51 a.m.) > > > Review request for Calligra. > > > Description > ------- > > At the sprint it was suggested to have this feature/bug fix in 2.4. Therefore > positing for review. It adds one new string. > > This add support for markers to calligra. > > Loading, saving, manipulation, work without problems > > At the moment the document loaded need to have markers so that the user can > select a marker on a path shape. I'm working on adding markers per default to > the docker at the moment. > > > This addresses bugs 260421, 260423 and 260431. > http://bugs.kde.org/show_bug.cgi?id=260421 > http://bugs.kde.org/show_bug.cgi?id=260423 > http://bugs.kde.org/show_bug.cgi?id=260431 > > > Diffs > ----- > > libs/flake/CMakeLists.txt 5face45 > libs/flake/KoDocumentResourceManager.h 2d6c8c0 > libs/flake/KoLineBorder.h 7f7d088 > libs/flake/KoLineBorder.cpp a2f0645 > libs/flake/KoMarker.h PRE-CREATION > libs/flake/KoMarker.cpp PRE-CREATION > libs/flake/KoMarkerCollection.h PRE-CREATION > libs/flake/KoMarkerCollection.cpp PRE-CREATION > libs/flake/KoMarkerData.h PRE-CREATION > libs/flake/KoMarkerData.cpp PRE-CREATION > libs/flake/KoMarkerSharedLoadingData.h PRE-CREATION > libs/flake/KoMarkerSharedLoadingData.cpp PRE-CREATION > libs/flake/KoPathShape.h 5b5b91b > libs/flake/KoPathShape.cpp 1898862 > libs/flake/KoPathShapeFactory.cpp f3a98b8 > libs/flake/KoPathShape_p.h e1e2843 > libs/flake/KoShapeLoadingContext.cpp 9d3d1da > libs/flake/KoShapeSavingContext.h a06e040 > libs/flake/KoShapeSavingContext.cpp 34f55c8 > libs/flake/commands/KoPathShapeMarkerCommand.h PRE-CREATION > libs/flake/commands/KoPathShapeMarkerCommand.cpp PRE-CREATION > libs/widgets/CMakeLists.txt 473a264 > libs/widgets/KoMarkerItemDelegate.h PRE-CREATION > libs/widgets/KoMarkerItemDelegate.cpp PRE-CREATION > libs/widgets/KoMarkerModel.h PRE-CREATION > libs/widgets/KoMarkerModel.cpp PRE-CREATION > libs/widgets/KoMarkerSelector.h PRE-CREATION > libs/widgets/KoMarkerSelector.cpp PRE-CREATION > libs/widgets/KoStrokeConfigWidget.h 3ae44b9 > libs/widgets/KoStrokeConfigWidget.cpp 9b2532f > marker_todo.txt PRE-CREATION > plugins/dockers/strokedocker/StrokeDocker.h dde8db7 > plugins/dockers/strokedocker/StrokeDocker.cpp 82f18b5 > > Diff: http://git.reviewboard.kde.org/r/103152/diff/diff > > > Testing > ------- > > Tested various documents with markers and they all worked without problems. > Tested manipulation of path shapes that have a marker attached. > > > Thanks, > > Thorsten Zachmann > >
_______________________________________________ calligra-devel mailing list calligra-devel@kde.org https://mail.kde.org/mailman/listinfo/calligra-devel
