----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118532/#review61679 -----------------------------------------------------------
Ship it! This looks good. I didn't look through the whole patch, I only used the new feature in the review board to look at the diff between the last two versions. And everything there looked fine except I think there is a missing comment. libs/kotext/commands/RenameSectionCommand.h <https://git.reviewboard.kde.org/r/118532/#comment42927> What is this variable for? I can understand the rest of the members without a comment but not this one. - Inge Wallin On July 4, 2014, 2:32 p.m., Denis Kuplyakov wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/118532/ > ----------------------------------------------------------- > > (Updated July 4, 2014, 2:32 p.m.) > > > Review request for Calligra, Camilla Boemann, Inge Wallin, and Dan Leinir > Turthra Jensen. > > > Repository: calligra > > > Description > ------- > > This adds proper section handling to code. It concentrates on managing to not > break section structure during editing of text. > Patch fixes many things: > 1) Insertion of text into sections > 2) Deletion of text with sections > 3) Cut, paste of text with sections > 4) Applying style doesn't remove section info > 5) WORDS product set added for fast building of Words and Author > 6) Sections visual indication (need improvements) > > Future TODOS: > 1) improving section indication (DONE) > 2) ability to insert section (DONE) > 3) adding unittests > 4) change icon for "New section" functionality (DONE) > 5) we should read "Undo New section" in edit, after inserting new one (DONE) > 6) fix indication of sections for multipage documents (DONE) > > > Diffs > ----- > > libs/flake/KoCanvasResourceManager.h 2ad1c6f > libs/flake/KoShapeLoadingContext.h 1e7d196 > libs/flake/KoShapeLoadingContext.cpp 65c9cbe > libs/flake/KoShapePaintingContext.h a7d15fd > libs/flake/KoShapePaintingContext.cpp 28ce859 > libs/kotext/CMakeLists.txt 44fa937 > libs/kotext/KoSection.h ac3fdd8 > libs/kotext/KoSection.cpp ca414b0 > libs/kotext/KoSectionEnd.h PRE-CREATION > libs/kotext/KoSectionEnd.cpp PRE-CREATION > libs/kotext/KoSectionManager.h PRE-CREATION > libs/kotext/KoSectionManager.cpp PRE-CREATION > libs/kotext/KoSectionUtils.h PRE-CREATION > libs/kotext/KoSectionUtils.cpp PRE-CREATION > libs/kotext/KoTextDocument.h 53c3d62 > libs/kotext/KoTextDocument.cpp 1cb0931 > libs/kotext/KoTextEditor.h 1fe289b > libs/kotext/KoTextEditor.cpp 574e803 > libs/kotext/KoTextEditor_format.cpp a82352d > libs/kotext/KoTextPaste.cpp de89bd8 > libs/kotext/commands/DeleteCommand.h f9dd2ef > libs/kotext/commands/DeleteCommand.cpp 391495e > libs/kotext/commands/RenameSectionCommand.h PRE-CREATION > libs/kotext/commands/RenameSectionCommand.cpp PRE-CREATION > libs/kotext/opendocument/KoTextLoader.h ca92573 > libs/kotext/opendocument/KoTextLoader.cpp bf3044e > libs/kotext/opendocument/KoTextWriter_p.cpp c03cebb > libs/kundo2/kundo2magicstring.h a911141 > libs/odf/KoOdfPaste.cpp 1e46546 > libs/textlayout/KoTextDocumentLayout.h 45a9ccb > libs/textlayout/KoTextLayoutArea.h f1b900a > libs/textlayout/KoTextLayoutArea_paint.cpp 2982c60 > plugins/textshape/CMakeLists.txt f7fcdb4 > plugins/textshape/TextShape.cpp 3b25ffe > plugins/textshape/TextTool.h 3409292 > plugins/textshape/TextTool.cpp ae51513 > plugins/textshape/dialogs/SectionFormatDialog.h PRE-CREATION > plugins/textshape/dialogs/SectionFormatDialog.cpp PRE-CREATION > plugins/textshape/dialogs/SectionFormatDialog.ui PRE-CREATION > plugins/textshape/dialogs/SimpleInsertWidget.cpp c2f083f > plugins/textshape/dialogs/SimpleInsertWidget.ui 4b9c639 > words/part/CMakeLists.txt 41f675b > words/part/KWApplicationConfig.h 87f1016 > words/part/KWApplicationConfig.cpp 1df6557 > words/part/KWDocument.h 4bc53e1 > words/part/KWDocument.cpp 443de3b > words/part/KWFactory.cpp 9894f40 > words/part/KWOdfLoader.cpp 4f82faa > words/part/KWView.h 50ee760 > words/part/KWView.cpp 6456d1f > words/part/dockers/KWDebugDocker.h PRE-CREATION > words/part/dockers/KWDebugDocker.cpp PRE-CREATION > words/part/dockers/KWDebugDockerFactory.h PRE-CREATION > words/part/dockers/KWDebugDockerFactory.cpp PRE-CREATION > words/part/dockers/KWDebugWidget.h PRE-CREATION > words/part/dockers/KWDebugWidget.cpp PRE-CREATION > words/part/words.rc c3a9822 > > Diff: https://git.reviewboard.kde.org/r/118532/diff/ > > > Testing > ------- > > I have tested all functionality with emphasis on deletion. It has the most > complicated code and need very good testing. > You can use LibreOffice Writer to create document with sections and then test > editing it in words. And I will attach one I've used. > > > File Attachments > ---------------- > > File for testing > > https://git.reviewboard.kde.org/media/uploaded/files/2014/06/04/f87d4ce2-3ab9-48a4-a880-4caa586790fc__Test-orig.odt > > > Thanks, > > Denis Kuplyakov > >
_______________________________________________ calligra-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/calligra-devel
