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

(Updated Jan. 8, 2011, 8:13 p.m.)


Review request for Calligra.


Changes
-------

Added screen shots of a test .doc with a numbered list displayed in msword and 
words before and after patching (1-3).

Note: A single text style is used to format the number of each list-item of a 
numbered list.  If a list-item has a different text style, then we can't format 
it's number like MSWord and OpenOffice do.  Check screen shot 4 and 5.


Summary
-------

Filter changes:
* Let's use the style-name attribute of a list-level-style-* to point to a text 
style, which should be used to format the bullet/number of a list.
* The style is stored into office:styles to be accessible via the style manager 
during layout.

KoListLevelProperties changes:
* Lookup the referred style using the style manager and update the 
CharacterStyleId of the KoListStyle.

TODO:
* If this approach is correct, also other filters should be updated.
* Bullet sizes! 


Diffs (updated)
-----

  filters/words/msword-odf/texthandler.h 1465e66 
  filters/words/msword-odf/texthandler.cpp b7d582c 
  libs/kotext/styles/KoListLevelProperties.cpp ec5a472 

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


Testing
-------

Using words application only, msword test documents containing numbered lists.


Screenshots
-----------

msword 2007
  http://git.reviewboard.kde.org/r/100306/s/33/
words
  http://git.reviewboard.kde.org/r/100306/s/34/
words patched
  http://git.reviewboard.kde.org/r/100306/s/35/
msword (list item sizes)
  http://git.reviewboard.kde.org/r/100306/s/36/
words (list item sizes)
  http://git.reviewboard.kde.org/r/100306/s/37/


Thanks,

Matus

_______________________________________________
calligra-devel mailing list
calligra-devel@kde.org
https://mail.kde.org/mailman/listinfo/calligra-devel

Reply via email to