Review Request 110910: Refactored context menu creation code out of KoResourceItemChooser and into its own QMenu class

Sascha Suelzer s_suelzer at lavabit.com
Sun Jun 9 11:35:51 BST 2013


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

Review request for Calligra.


Description
-------

Refactored context menu creation code out of KoResourceItemChooser and into its own QMenu class, any style feedback is welcome.
Also a tiny amount of whitespace cleanup, but only for the files affected by the code changes.


Diffs
-----

  libs/widgets/KoResourceItemChooser.h 29fb09a 
  libs/widgets/KoResourceItemChooser.cpp 4ae16ad 
  libs/widgets/KoResourceItemChooserContextMenu.h a33a132 
  libs/widgets/KoResourceItemChooserContextMenu.cpp 6af2fe1 

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


Testing
-------

Tested only for Krita, everything seems to work as it should.


Thanks,

Sascha Suelzer

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


More information about the calligra-devel mailing list