Review Request 109651: Refactor style manager

Commit Hook null at kde.org
Tue Mar 26 17:40:33 GMT 2013


-----------------------------------------------------------
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.)


Status
------

This change has been marked as submitted.


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

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20130326/6eeff98a/attachment.htm>


More information about the calligra-devel mailing list