> On Jan. 24, 2015, 8:23 p.m., Inge Wallin wrote: > > libs/kotext/commands/ChangeStylesCommand.h, line 38 > > <https://git.reviewboard.kde.org/r/122238/diff/1/?file=344595#file344595line38> > > > > This seems strange to me. Why would a style change store a single > > QTextDocument? An ODF style change affects all the ODF document with > > several QTextDocuments.
because this class handles it for a single document. The ChangeStylesMacroCommand hold one such sub-command per document :) - Camilla ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122238/#review74674 ----------------------------------------------------------- On Jan. 24, 2015, 7:27 p.m., Camilla Boemann wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/122238/ > ----------------------------------------------------------- > > (Updated Jan. 24, 2015, 7:27 p.m.) > > > Review request for Calligra. > > > Repository: calligra > > > Description > ------- > > Make KoStyleManager independent of rest of libs/kotext - making it possible > to move it to libs/odf > also fixes a bug that stylechanges were never applied to the document > > > Diffs > ----- > > libs/kotext/CMakeLists.txt bfb0b2d > libs/kotext/KoTextDocument.cpp 07a47dc > libs/kotext/OdfTextTrackStyles.h PRE-CREATION > libs/kotext/OdfTextTrackStyles.cpp PRE-CREATION > libs/kotext/commands/ChangeStylesCommand.h 3762715 > libs/kotext/commands/ChangeStylesCommand.cpp 771caae > libs/kotext/commands/ChangeStylesMacroCommand.h 340ba12 > libs/kotext/commands/ChangeStylesMacroCommand.cpp 66f5326 > libs/kotext/styles/ChangeFollower.h 5786541 > libs/kotext/styles/ChangeFollower.cpp 7930cfe > libs/kotext/styles/KoStyleManager.h e1b400c > libs/kotext/styles/KoStyleManager.cpp 86108d6 > libs/textlayout/KoTextShapeData.cpp c51ec6e > sheets/Map.cpp a4fcc7c > > Diff: https://git.reviewboard.kde.org/r/122238/diff/ > > > Testing > ------- > > only basic editing and seeing that undo/redo works and that changes are now > actually appied to the document > > > Thanks, > > Camilla Boemann > >
_______________________________________________ calligra-devel mailing list calligra-devel@kde.org https://mail.kde.org/mailman/listinfo/calligra-devel
