Review Request: adds a VectorShapeConfigWidget which gets shown on shape creation (like with PictureShape)

[Friedrich W. H. Kossebau]

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

Review request for Calligra and Inge Wallin.


Description
-------

Other than with the picture shape, after adding a new vector shape one first has to activate the vector tool, then click the open button.
Not very user-friendly and also inconsistent -> isn't that a bug from your, Inge's, POV? :) (\me thinks of zooming behaviour in Stage).

Attached patch fixes this by adding a configwidget also to the vector shape, copied from the picture shape and adapted.
The patch also fixes the icon for the shape and the tool to an existing one.
And adds "(EMF/WMF)" to the tooltip of the shape, to make it more obvious what this shape is about (especially in Karbon).

Q1: Is that vector shape just for EMF, WMF officially? What about SVG (not only as in SVM from OOo)?

Q2: Seems there is no official mimetype for WMF and EMF. Wikipedia say image/x-wmf and image/x-emf, and so does the XDG database (that's why these are used for the filedialog, while Oxygen icons use application-x-wmf, like the code in VectorShape.cpp with application/x-wmf and application/x-emf. Is the code correct here?

Okay to backport to 2.5, without the tooltip string change?


Diffs
-----

  plugins/vectorshape/CMakeLists.txt da25dbb 
  plugins/vectorshape/VectorShapeConfigWidget.h PRE-CREATION 
  plugins/vectorshape/VectorShapeConfigWidget.cpp PRE-CREATION 
  plugins/vectorshape/VectorShapeFactory.cpp 07e02ae 
  plugins/vectorshape/VectorToolFactory.cpp 04212c6 

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


Testing
-------


Thanks,

Friedrich W. H. Kossebau

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