----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/109651/#review29977 -----------------------------------------------------------
This review has been submitted with commit d91e0681b7d43eb3c91a0e7fe5051e3a93fa2386 by Thorsten Zachmann to branch calligra/2.6. - Commit Hook On March 26, 2013, 5:40 p.m., Thorsten Zachmann wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/109651/ > ----------------------------------------------------------- > > (Updated March 26, 2013, 5:40 p.m.) > > > Review request for Calligra and Pierre Stirnweiss. > > > Description > ------- > > This patch tries to fix the problems/crashes which happen in the style > manager. > > To get the simple crash open the default document and click on the edit > button of the None (character style) > > The patch uses a different model for the style manager and temporary styles > that get modified to only apply the changes once the user applies them. > The model changes the internal pointer to the temporary files once they are > selected and when applied resets to the original ones. > However the styles preview in the styles manager list is updated immediately. > The patch simplifies the handling quite a bit. > > I did not know that Pierre was also working on that so I publish it now so we > have something we can discuss. > > If you like this I would like to backport it to 2.6 as it fixes quite some > grave bugs compared to what we have now. > > > Diffs > ----- > > libs/kotext/styles/KoCharacterStyle.h 2fdde12 > libs/textlayout/KoStyleThumbnailer.cpp 436b243 > plugins/textshape/CMakeLists.txt 084628c > plugins/textshape/dialogs/CharacterGeneral.h e6d99db > plugins/textshape/dialogs/CharacterGeneral.cpp d3eead6 > plugins/textshape/dialogs/ParagraphGeneral.h 1a78376 > plugins/textshape/dialogs/ParagraphGeneral.cpp ad30369 > plugins/textshape/dialogs/StyleManager.h dec3b6b > plugins/textshape/dialogs/StyleManager.cpp 9c82c11 > plugins/textshape/dialogs/StylesManagerModel.h PRE-CREATION > plugins/textshape/dialogs/StylesManagerModel.cpp PRE-CREATION > plugins/textshape/dialogs/StylesSortFilterProxyModel.h PRE-CREATION > plugins/textshape/dialogs/StylesSortFilterProxyModel.cpp PRE-CREATION > > Diff: http://git.reviewboard.kde.org/r/109651/diff/ > > > Testing > ------- > > Tested modification without saving. > Tested applying. > Tested Ok. > Tested Canceling > > > Thanks, > > Thorsten Zachmann > >
_______________________________________________ calligra-devel mailing list calligra-devel@kde.org https://mail.kde.org/mailman/listinfo/calligra-devel
