Review Request: Changing from QMaps to QLists in createOptionWidgets()

Samuel Buttigieg sambut1987 at gmail.com
Mon May 23 20:19:14 BST 2011


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

(Updated May 23, 2011, 7:19 p.m.)


Review request for Calligra.


Changes
-------

round 3


Summary
-------

As requested by boemann, this patch replaces all instances of QMaps used to represent Widgets with QLists. Using QLists allows the widgets to be displayed in the order you like them to.


Diffs (updated)
-----

  braindump/plugins/stateshape/StateTool.h 7ab14cb 
  braindump/plugins/stateshape/StateTool.cpp 79d7927 
  braindump/plugins/webshape/WebTool.h 2998d09 
  braindump/plugins/webshape/WebTool.cpp 44901ae 
  karbon/plugins/tools/KarbonGradientTool.h bb87f8f 
  karbon/plugins/tools/KarbonGradientTool.cpp 02718b0 
  karbon/plugins/tools/KarbonPatternTool.h f9792a0 
  karbon/plugins/tools/KarbonPatternTool.cpp 51e57dd 
  karbon/plugins/tools/filterEffectTool/KarbonFilterEffectsTool.h a25b54e 
  karbon/plugins/tools/filterEffectTool/KarbonFilterEffectsTool.cpp 1d02cca 
  karbon/ui/KarbonView.cpp c5b682b 
  kpresenter/part/tools/animationtool/KPrAnimationTool.h 117bc61 
  kpresenter/part/tools/animationtool/KPrAnimationTool.cpp 0a1b997 
  krita/plugins/tools/defaulttools/kis_tool_path.h f57a1fa 
  krita/plugins/tools/defaulttools/kis_tool_path.cc 80a31cf 
  krita/plugins/tools/selectiontools/kis_tool_select_path.h 7bb8a14 
  krita/plugins/tools/selectiontools/kis_tool_select_path.cc cd4c40d 
  krita/ui/kis_view2.cpp bd0d82f 
  libs/flake/KoCanvasControllerWidget.h 384c928 
  libs/flake/KoCanvasControllerWidget.cpp 7da9e38 
  libs/flake/KoToolBase.h 860a77c 
  libs/flake/KoToolBase.cpp 456587d 
  libs/flake/KoToolBase_p.h 49bc2f9 
  libs/flake/KoToolManager.cpp bf68ad9 
  libs/flake/tools/KoCreatePathTool.h 325af3b 
  libs/flake/tools/KoCreatePathTool.cpp ca8f373 
  libs/flake/tools/KoPathTool.h bcd6d95 
  libs/flake/tools/KoPathTool.cpp 157d387 
  libs/kopageapp/KoPAView.cpp 141e46c 
  libs/kopageapp/tools/backgroundTool/KoPABackgroundTool.h 17a83b2 
  libs/kopageapp/tools/backgroundTool/KoPABackgroundTool.cpp 0b16048 
  libs/main/KoDockerManager.h 3ec84b9 
  libs/main/KoDockerManager.cpp 93f1b3e 
  libs/main/KoToolDocker.cpp c9aacce 
  libs/main/KoToolDocker_p.h 3f76df7 
  plugins/artistictextshape/ArtisticTextTool.h e9b6347 
  plugins/artistictextshape/ArtisticTextTool.cpp 68debef 
  plugins/defaultTools/connectionTool/ConnectionTool.h 09ece57 
  plugins/defaultTools/connectionTool/ConnectionTool.cpp e0d4594 
  plugins/defaultTools/defaulttool/DefaultTool.h 724798f 
  plugins/defaultTools/defaulttool/DefaultTool.cpp c9fb9ab 
  plugins/defaultTools/guidestool/GuidesTool.h 2cc8d00 
  plugins/defaultTools/guidestool/GuidesTool.cpp 635518b 
  plugins/textshape/ReferencesTool.h 7fb9424 
  plugins/textshape/ReferencesTool.cpp 47d1fbd 
  plugins/textshape/ReviewTool.h 90e8bf0 
  plugins/textshape/ReviewTool.cpp 2920ea8 
  plugins/textshape/TextTool.h 5460ba8 
  plugins/textshape/TextTool.cpp 44035a6 
  plugins/treeshape/TreeTool.h 0c8733c 
  plugins/treeshape/TreeTool.cpp 5c26dc9 
  tables/part/View.cpp 82bbcbf 
  tables/shape/TableTool.h 45a9ab1 
  tables/shape/TableTool.cpp 1d54e92 
  tables/ui/CellToolBase.h 1e85233 
  tables/ui/CellToolBase.cpp 3557fb1 
  words/part/KWGui.cpp 7ec4d16 

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


Testing
-------

checking if user interface still works and making sure no crashes occur


Thanks,

Samuel

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


More information about the calligra-devel mailing list