> On Oct. 1, 2011, 5:09 a.m., Thorsten Zachmann wrote: > > Looks good. Please commit after fixing the pointed out. > > > > I'm not sure about the new classname KoShapeBasedDocumentBase. The > > orrurence of two times Base somehow troubles me. How about renaming it to > > KoShapeDocumentBase? That is just my though and it is nothing that needs to > > be done before committing this.
Well, it's a base class for documents that are based on shapes -- and not all KoDocument-based documents are also necessarily based on shapes, so it's Ko - Shape-Based - Document - Base class. It's ugly, but the clearest we could come up with on irc when disgcussing this change. - Boudewijn ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102679/#review6979 ----------------------------------------------------------- On Sept. 29, 2011, 10:16 a.m., Boudewijn Rempt wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/102679/ > ----------------------------------------------------------- > > (Updated Sept. 29, 2011, 10:16 a.m.) > > > Review request for Calligra. > > > Description > ------- > > KoTextEditor is meant to be the one and only interface we allow to editing a > QTextDocument. TextTool breaks this encapsulation in many ways. This patch > improves the situation but doesn't solve it completely yet. Several commands > have been moved to kotext and encapsulated in KoTextEditor. This simplifies > the code in the textshape quite a bit. The other bits will follow later on. > > In order to make it possible to test this code, I wanted to be able to create > a KoShapeController without a canvas, so KoShapeController was refactored a > bit as well. Because KoShapeControllerBase was confusingly named, I renamed > that class after irc discussion. > > > Diffs > ----- > > karbon/common/commands/KarbonBooleanCommand.h 9c49a74 > karbon/common/commands/KarbonBooleanCommand.cpp 947ba83 > karbon/ui/KarbonPart.h 2dc5f85 > karbon/ui/dockers/KarbonLayerDocker.h 55ee45d > karbon/ui/dockers/KarbonLayerDocker.cpp 30e1ab2 > karbon/ui/dockers/KarbonLayerModel.cpp f26666f > libs/flake/CMakeLists.txt 4311bd0 > libs/flake/KoCanvasBase.h 5f8f0ab > libs/flake/KoCanvasBase.cpp 2361bc1 > libs/flake/KoCanvasController.h 3fc370e > libs/flake/KoDataCenterBase.h de447fa > libs/flake/KoResourceManager.h f8fd13c > libs/flake/KoShape.h 84bdbfc > libs/flake/KoShapeBasedDocumentBase.h PRE-CREATION > libs/flake/KoShapeBasedDocumentBase.cpp PRE-CREATION > libs/flake/KoShapeController.h e3c65ab > libs/flake/KoShapeController.cpp 27722b3 > libs/flake/KoShapeControllerBase.h 5d1db1f > libs/flake/KoShapeControllerBase.cpp 3f9d9a2 > libs/flake/KoShapeLoadingContext.h b0e0358 > libs/flake/KoShapeLoadingContext.cpp 50d50bb > libs/flake/KoShapePaste.cpp d94dc11 > libs/flake/KoToolBase.h 13dc3c7 > libs/flake/KoToolBase.cpp 627591e > libs/flake/KoToolManager.h 48cbaa8 > libs/flake/KoToolManager.cpp 8c612b0 > libs/flake/commands/KoPathCombineCommand.h a34edb5 > libs/flake/commands/KoPathCombineCommand.cpp 72aaed4 > libs/flake/commands/KoPathPointRemoveCommand.cpp fa748a6 > libs/flake/commands/KoPathSeparateCommand.h 4b52a60 > libs/flake/commands/KoPathSeparateCommand.cpp a7b0ab7 > libs/flake/commands/KoShapeClipCommand.h 883402c > libs/flake/commands/KoShapeClipCommand.cpp 7159c91 > libs/flake/commands/KoShapeCreateCommand.h 4a9f3b8 > libs/flake/commands/KoShapeCreateCommand.cpp 6322c8d > libs/flake/commands/KoShapeDeleteCommand.h 1d1eac4 > libs/flake/commands/KoShapeDeleteCommand.cpp 452ffea > libs/flake/commands/KoShapeUnclipCommand.h 8903e33 > libs/flake/commands/KoShapeUnclipCommand.cpp 94d2308 > libs/flake/tests/MockShapes.h 3f7ba47 > libs/flake/tests/TestSnapStrategy.cpp 7f2b63b > libs/kopageapp/KoPADocument.h cfc9822 > libs/kopageapp/KoPADocumentModel.cpp b968182 > libs/kopageapp/KoPADocumentStructureDocker.cpp 15c3224 > libs/kotext/CMakeLists.txt b174f5d > libs/kotext/KoDocumentRdfBase.h 165f8fd > libs/kotext/KoDocumentRdfBase.cpp f6445e6 > libs/kotext/KoTextCommandBase.h PRE-CREATION > libs/kotext/KoTextCommandBase.cpp PRE-CREATION > libs/kotext/KoTextDocument.cpp 4d8c992 > libs/kotext/KoTextEditor.h 7a51591 > libs/kotext/KoTextEditor.cpp 9ff740c > libs/kotext/KoTextOdfSaveHelper.h 4e247c2 > libs/kotext/KoTextOdfSaveHelper.cpp 9d5add8 > libs/kotext/KoTextPaste.h 61321d9 > libs/kotext/KoTextPaste.cpp 067d238 > libs/kotext/commands/ChangeListCommand.h PRE-CREATION > libs/kotext/commands/ChangeListCommand.cpp PRE-CREATION > libs/kotext/commands/ChangeTrackedDeleteCommand.h PRE-CREATION > libs/kotext/commands/ChangeTrackedDeleteCommand.cpp PRE-CREATION > libs/kotext/commands/DeleteCommand.h PRE-CREATION > libs/kotext/commands/DeleteCommand.cpp PRE-CREATION > libs/kotext/commands/ListItemNumberingCommand.h PRE-CREATION > libs/kotext/commands/ListItemNumberingCommand.cpp PRE-CREATION > libs/kotext/commands/TextPasteCommand.h PRE-CREATION > libs/kotext/commands/TextPasteCommand.cpp PRE-CREATION > libs/kotext/opendocument/KoTextWriter.h 04ea489 > libs/kotext/opendocument/KoTextWriter.cpp 5d530b7 > libs/kotext/tests/TestKoTextEditor.h 8013086 > libs/kotext/tests/TestKoTextEditor.cpp 85fab42 > libs/main/rdf/KoDocumentRdf.h 219ff42 > libs/main/rdf/KoDocumentRdf.cpp bc5322a > libs/pigment/KoColorSpaceTraits.h 4bdd16b > plugins/dockers/shapecollection/CollectionShapeFactory.h 870fe2e > plugins/dockers/shapecollection/CollectionShapeFactory.cpp 020e1af > plugins/pictureshape/PictureShapeFactory.cpp 9ae730c > plugins/pluginshape/PluginShapeFactory.cpp ee5f508 > plugins/textshape/CMakeLists.txt 87d59f4 > plugins/textshape/TextShapeFactory.cpp 4100b47 > plugins/textshape/TextTool.h 6aaef61 > plugins/textshape/TextTool.cpp 142d934 > plugins/textshape/commands/AcceptChangeCommand.h 2945d9e > plugins/textshape/commands/AcceptChangeCommand.cpp 66b121c > plugins/textshape/commands/ChangeListCommand.h a7c2f7e > plugins/textshape/commands/ChangeListCommand.cpp 8981a1b > plugins/textshape/commands/ChangeListLevelCommand.h f657ee1 > plugins/textshape/commands/ChangeListLevelCommand.cpp bad0b68 > plugins/textshape/commands/ChangeTrackedDeleteCommand.h 6acf4bd > plugins/textshape/commands/ChangeTrackedDeleteCommand.cpp f155681 > plugins/textshape/commands/DeleteCommand.h b85bbb9 > plugins/textshape/commands/DeleteCommand.cpp cd741dc > plugins/textshape/commands/ListItemNumberingCommand.h 4457d84 > plugins/textshape/commands/ListItemNumberingCommand.cpp f00162b > plugins/textshape/commands/RejectChangeCommand.h 925138d > plugins/textshape/commands/RejectChangeCommand.cpp 3338875 > plugins/textshape/commands/ShowChangesCommand.h cacd86a > plugins/textshape/commands/ShowChangesCommand.cpp e61f883 > plugins/textshape/commands/TextCommandBase.h d6306db > plugins/textshape/commands/TextCommandBase.cpp be52032 > plugins/textshape/commands/TextCutCommand.cpp 31776f9 > plugins/textshape/commands/TextPasteCommand.h 90f4c3d > plugins/textshape/commands/TextPasteCommand.cpp 36a1f76 > plugins/textshape/dialogs/ParagraphSettingsDialog.cpp cdddbe3 > plugins/textshape/dialogs/SimpleParagraphWidget.cpp 5b843ba > plugins/textshape/tests/CMakeLists.txt e6ab42a > plugins/videoshape/VideoShapeFactory.cpp f1c8d79 > stage/part/KPrPlaceholderStrategy.h e5ea2cb > words/part/KWDocument.h 4143803 > words/part/commands/KWFrameCreateCommand.h 354486d > words/part/commands/KWFrameCreateCommand.cpp b4a4fb1 > words/part/commands/KWFrameDeleteCommand.h 1bc79b8 > words/part/commands/KWFrameDeleteCommand.cpp c5d3c4b > > Diff: http://git.reviewboard.kde.org/r/102679/diff/diff > > > Testing > ------- > > manual gui test + ran the unittests. Added more testing. > > > Thanks, > > Boudewijn Rempt > >
_______________________________________________ calligra-devel mailing list calligra-devel@kde.org https://mail.kde.org/mailman/listinfo/calligra-devel
