Review Request 112080: Refactor tag widgets and tag logic out of KoResourceItemchooser into their own classes and widgets

Sascha Suelzer s.suelzer at gmail.com
Thu Aug 15 06:48:58 BST 2013


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

(Updated Aug. 15, 2013, 5:48 a.m.)


Review request for Calligra.


Changes
-------

Fix splitter layout gaps.


Description (updated)
-------

A rather chunky patch again, I'm afraid.
KoResourceItemChooser has been stripped of the tagging code, along with the GUI bits for it.
KoResourceTaggingInterface now contains the logic (assignment, renaming, etc), it also does the context menu for the resources themselves (maybe that should go into its own bit, too).
KoTagFilterWidget is the GUI bit for all filtering/searching operations.
KoTagChooserWidget is the GUI bit for choosing tags along with a tool button for modifying them (addition, removal, renaming, undeletion)

I've also updated my email address as well as some whitespace errors which got fixed along the way.

------------------------------------------
This is my first attempt at creating custom widgets.
That is most likely the reason for several issues that the code has.

1.) I have no idea if the widget/GUI code I have written is up to style/standards in general. (pimpl (which I was corrected on before, but I think it makes sense this time), using privacy and friends, etc) 
2.) I don't know if I split the components sensibly.

Any pointers on any of that will be highly appreciated and incorporated as promptly as I am able to ^^ 


Diffs (updated)
-----

  krita/ui/kis_palette_manager.cpp 53b9c58 
  krita/ui/widgets/kis_preset_chooser.h 92d1501 
  krita/ui/widgets/kis_preset_chooser.cpp 4da9ed3 
  krita/ui/widgets/kis_workspace_chooser.cpp 0100372 
  libs/widgets/CMakeLists.txt a1b9344 
  libs/widgets/KoResourceFiltering.h ca5a360 
  libs/widgets/KoResourceFiltering.cpp 4a3976e 
  libs/widgets/KoResourceItemChooser.h b5e0022 
  libs/widgets/KoResourceItemChooser.cpp 2f9e71f 
  libs/widgets/KoResourceItemChooserContextMenu.h fb5bfbb 
  libs/widgets/KoResourceItemChooserContextMenu.cpp bfeb907 
  libs/widgets/KoResourceModel.h 783de22 
  libs/widgets/KoResourceModel.cpp 636dd44 
  libs/widgets/KoResourceServer.h a56e9b4 
  libs/widgets/KoResourceServerAdapter.h 7d63504 
  libs/widgets/KoResourceTaggingInterface.h PRE-CREATION 
  libs/widgets/KoResourceTaggingInterface.cpp PRE-CREATION 
  libs/widgets/KoTagChooserWidget.h PRE-CREATION 
  libs/widgets/KoTagChooserWidget.cpp PRE-CREATION 
  libs/widgets/KoTagFilterWidget.h PRE-CREATION 
  libs/widgets/KoTagFilterWidget.cpp PRE-CREATION 

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


Testing
-------

Manual, thorough testing, Krita only. Could not detect regressions in actual functionality.


Thanks,

Sascha Suelzer

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


More information about the calligra-devel mailing list