----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104047/#review10894 -----------------------------------------------------------
Ship it! I'm approving it, since it is just as much pierrest's work as my own. And since I have not found any problems with it please object if you think you would have more insight and be able to do a quality review - C. Boemann On Feb. 24, 2012, 12:32 p.m., C. Boemann wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/104047/ > ----------------------------------------------------------- > > (Updated Feb. 24, 2012, 12:32 p.m.) > > > Review request for Calligra. > > > Description > ------- > > We have made changes to the undo framework in kotexteditor to make things > work. > The DeleteCommand has become much more capable as it handles complex > selections and bookmarks now > The KoTextEditor.cpp file has been split out into 3 files > The StyleChange commands and shapeAnchoring commands have been adapted to the > new frame work > > > Diffs > ----- > > libs/flake/KoShapeController.h abc4358 > libs/kotext/CMakeLists.txt 567649b > libs/kotext/KoTextDocument.h 56f55e6 > libs/kotext/KoTextDocument.cpp 5608860 > libs/kotext/KoTextEditor.h 689051b > libs/kotext/KoTextEditor.cpp 3c141c7 > libs/kotext/KoTextEditor_format.cpp PRE-CREATION > libs/kotext/KoTextEditor_p.h 77725f3 > libs/kotext/KoTextEditor_undo.cpp PRE-CREATION > libs/kotext/commands/ChangeAnchorPropertiesCommand.h 4206bb8 > libs/kotext/commands/ChangeAnchorPropertiesCommand.cpp 7d4b912 > libs/kotext/commands/ChangeStylesCommand.h accfd31 > libs/kotext/commands/ChangeStylesCommand.cpp b9fa725 > libs/kotext/commands/ChangeStylesMacroCommand.cpp 34c3822 > libs/kotext/commands/ChangeTrackedDeleteCommand.cpp 89edde1 > libs/kotext/commands/DeleteAnchorsCommand.cpp 621829f > libs/kotext/commands/DeleteCommand.h 89b448a > libs/kotext/commands/DeleteCommand.cpp 2a4527d > libs/kotext/commands/TextPasteCommand.cpp 6b9de68 > libs/kotext/tests/TestKoTextEditor.cpp bd15b3a > libs/kundo2/kundo2stack.h c0539fe > libs/kundo2/kundo2stack.cpp 48d6625 > libs/main/rdf/KoSemanticStylesheet.cpp 7853dd0 > libs/main/tests/rdf_test.cpp fb57a5e > plugins/textshape/TextTool.cpp ddfe47f > plugins/textshape/commands/TextCutCommand.cpp bdb8eea > words/part/KWDocument.h 0005bc6 > words/part/KWDocument.cpp 8e00cb6 > words/part/dialogs/KWAnchoringProperties.cpp c742e26 > words/part/dialogs/KWFrameDialog.cpp 0fcc0b7 > words/part/frames/KWTextFrameSet.cpp ebd532f > > Diff: http://git.reviewboard.kde.org/r/104047/diff/ > > > Testing > ------- > > We have played around with undo redo in all the cases we knew were failing > and quite a lot of other tests. > Undo of many things are still not implemented, but that is not really what > this is about. It's about the basic undo, and getting the framework right. > Fixing undo of individual stuff can always be done as bug fixes > One issue we have noted is insertion of table, whic we believe is a qt bug. > We have one last workaround in our minds, but other than that we will just > have to wait for a qt fix > > > Thanks, > > C. Boemann > >
_______________________________________________ calligra-devel mailing list calligra-devel@kde.org https://mail.kde.org/mailman/listinfo/calligra-devel
