Review Request: Replace the current half baked style selection widgets with a home brewed comboBox

Pierre Stirnweiss pstirnweiss at googlemail.com
Sun Jan 15 19:41:46 GMT 2012


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/103677/
-----------------------------------------------------------

(Updated Jan. 15, 2012, 7:41 p.m.)


Review request for Calligra.


Changes
-------

Remove patch files from branch


Description
-------

Merge request.
Merge branch textshape_stylesWidget_pierreSt into master.
The branch provides a replacement for the style selection widgets. The new provided widgets are a subclass of QComboBox.
They provide a style preview both in the main area and in the dropdown menu.
The dropdown menu provides a "show style manager" button, which call the whole style management dialog.
The dropdown also provides a "delete style" button, although this is for the moment disabled/commented out, because deleting a style is not currently really supported.
The combo also allows to create a style based on the current text format if it does not correspond to the original style (ie. the user changed a property).


Diffs (updated)
-----

  libs/kotext/KoTextEditor.h 0fc4ff4 
  libs/kotext/KoTextEditor.cpp 3ef31d6 
  libs/kotext/styles/KoCharacterStyle.h 898ba43 
  libs/kotext/styles/KoCharacterStyle.cpp e575b0d 
  libs/kotext/styles/KoParagraphStyle.h 12ab672 
  libs/textlayout/CMakeLists.txt af6d64a 
  libs/textlayout/FrameIterator.h c73d580 
  libs/textlayout/KoStyleThumbnailer.h 8645089 
  libs/textlayout/KoStyleThumbnailer.cpp 3529bbb 
  plugins/textshape/CMakeLists.txt fad7c04 
  plugins/textshape/TextTool.h 17a687b 
  plugins/textshape/TextTool.cpp 3ba9d33 
  plugins/textshape/dialogs/SimpleCharacterWidget.h ad944a5 
  plugins/textshape/dialogs/SimpleCharacterWidget.cpp 6b2bb66 
  plugins/textshape/dialogs/SimpleCharacterWidget.ui e4f6d3b 
  plugins/textshape/dialogs/SimpleParagraphWidget.h bc167b4 
  plugins/textshape/dialogs/SimpleParagraphWidget.cpp cbfacfe 
  plugins/textshape/dialogs/SimpleParagraphWidget.ui b2e9adf 
  plugins/textshape/dialogs/StylesCombo.h PRE-CREATION 
  plugins/textshape/dialogs/StylesCombo.cpp PRE-CREATION 
  plugins/textshape/dialogs/StylesComboPreview.h PRE-CREATION 
  plugins/textshape/dialogs/StylesComboPreview.cpp PRE-CREATION 
  plugins/textshape/dialogs/StylesDelegate.h f5d216f 
  plugins/textshape/dialogs/StylesDelegate.cpp d1021f5 
  plugins/textshape/dialogs/StylesModel.h dbd17b4 
  plugins/textshape/dialogs/StylesModel.cpp 94deb24 
  plugins/textshape/dialogs/StylesWidget.cpp 4f31a01 
  plugins/textshape/dialogs/TableOfContentsStyleModel.cpp 48ee7c3 

Diff: http://git.reviewboard.kde.org/r/103677/diff/diff


Testing
-------

The functionality exclusively linked to the comboBox seems to be working properly. Some defects have been identified (like no paragraph style is selected at startup), but these are actually outside the scope of this widget and affecting the current widget also.


Thanks,

Pierre Stirnweiss

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


More information about the calligra-devel mailing list