Review Request: Improved html-odf Words export filter

Stuart Dickson stuartmd at kogmbh.com
Tue Jan 24 12:31:00 GMT 2012


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

(Updated Jan. 24, 2012, 12:31 p.m.)


Review request for Calligra.


Changes
-------

Updated patch to reflect first set of Review Board requests.

There may be an issue with this - the autogenerated patch included other commits, and these were removed from this file.


Description
-------

Improvements to the html-odf export filter.

Added support for: exporting images, tables with merged cells.
Modifications to KoFilter derived class to incorporate Conversion class' convert function and provide improved error checking.
Changes to output styles to improve legibility.
Disabled export filter dialog as it does not have an impact on output, but can be modified in future. 
Removing QChar::ObjectReplacementMarkers from appearing in output.

The focus has been on the content and not on styles and there is still improvement in cleaning up the generated output.


Diffs (updated)
-----

  filters/words/html-odf/CMakeLists.txt d1bee3f 
  filters/words/html-odf/ManifestParser.h PRE-CREATION 
  filters/words/html-odf/ManifestParser.cpp PRE-CREATION 
  filters/words/html-odf/conversion/converter.xsl 1f01ed6 
  filters/words/html-odf/convert.h 8cbdd6c 
  filters/words/html-odf/convert.cpp 226fbd5 
  filters/words/html-odf/htmlodfexport.h bff8dec 
  filters/words/html-odf/htmlodfexport.cpp 5f0606e 

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


Testing
-------

Testing against a couple of sample documents including paragraphs, titles, simple tables and ones with additional merged cells, inline and paragraph anchored images.


Thanks,

Stuart Dickson

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


More information about the calligra-devel mailing list