> On June 23, 2014, 4:08 a.m., Inge Wallin wrote: > > plugins/textshape/dialogs/SimpleInsertWidget.ui, line 40 > > <https://git.reviewboard.kde.org/r/118532/diff/4/?file=283759#file283759line40> > > > > Is this the only clue the user will get?
Nearest button in this file has the same thing in this property. I though it should be done this way for icon-only button. - Denis ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118532/#review60742 ----------------------------------------------------------- On June 23, 2014, 9:07 a.m., Denis Kuplyakov wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/118532/ > ----------------------------------------------------------- > > (Updated June 23, 2014, 9:07 a.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 > 5) we should read "Undo New section" in edit, after inserting new one > 6) fix indication of sections for multipage documents > > > Diffs > ----- > > cmake/productsets/words.cmake PRE-CREATION > libs/flake/KoCanvasResourceManager.h 2ad1c6f > 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/KoSectionUtils.h PRE-CREATION > libs/kotext/KoSectionUtils.cpp PRE-CREATION > 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/opendocument/KoTextLoader.h ca92573 > libs/kotext/opendocument/KoTextLoader.cpp bf3044e > libs/kotext/opendocument/KoTextWriter_p.cpp c03cebb > libs/kundo2/kundo2magicstring.h f60f2a7 > libs/odf/KoOdfPaste.cpp 1e46546 > libs/textlayout/KoTextDocumentLayout.h 45a9ccb > libs/textlayout/KoTextLayoutArea.h f1b900a > libs/textlayout/KoTextLayoutArea_paint.cpp 2982c60 > plugins/textshape/TextShape.cpp 3b25ffe > plugins/textshape/TextTool.h 3409292 > plugins/textshape/TextTool.cpp ae51513 > 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/KWFactory.cpp 9894f40 > 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 calligra-devel@kde.org https://mail.kde.org/mailman/listinfo/calligra-devel
