----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103152/#review8261 -----------------------------------------------------------
libs/flake/KoLineBorder.h <http://git.reviewboard.kde.org/r/103152/#comment7062> A small proposal. Can we start using passing via QPainter* recommended by Qt API design guidelines? This change would have to be adopted by other methods in this class and elsewhere but complying with the guideline in the new code would reduce amount of work in the future. The same note applies to other methods you added in this patch. libs/flake/KoMarker.h <http://git.reviewboard.kde.org/r/103152/#comment7063> Can the two methods be glued into one? + simple docs are missing libs/flake/KoMarker.cpp <http://git.reviewboard.kde.org/r/103152/#comment7064> I am afraid this expression does not give unique name. To make it unique one would have to escape '_' character as well. libs/flake/KoMarkerCollection.h <http://git.reviewboard.kde.org/r/103152/#comment7065> + explicit libs/flake/KoMarkerCollection.h <http://git.reviewboard.kde.org/r/103152/#comment7066> no need for virtual here... libs/flake/KoMarkerSharedLoadingData.h <http://git.reviewboard.kde.org/r/103152/#comment7067> const QHash & + explicit libs/flake/KoShapeSavingContext.cpp <http://git.reviewboard.kde.org/r/103152/#comment7068> const KoMarker & libs/flake/KoShapeSavingContext.cpp <http://git.reviewboard.kde.org/r/103152/#comment7069> it's easier to use: !d->markerRefs.contains(marker) - Jarosław Staniek 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
