Review Request: KoReport: fix save/load page margins

Dag Andersen danders at get2net.dk
Tue Jul 3 14:36:47 BST 2012


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

(Updated July 3, 2012, 1:36 p.m.)


Review request for Calligra, Jarosław Staniek and Adam Pigg.


Changes
-------

Updated diff:
Tested with and passes new unit tests


Description
-------

Seems page margins never worked:
* Saveing used namespace fo, loading used report
* Loading expected values in pt, saving was in localized value + unit

This changes:
* Saving to use namespace 'fo' and value in points.
* Loading to try to load any value + optional unit (with KoUnit::parseValue()).


Diffs (updated)
-----

  libs/koreport/common/KoReportItemBase.cpp 76dc9a8 
  libs/koreport/common/krreportdata.cpp 4adf547 
  libs/koreport/common/krutils.h 962e1e7 
  libs/koreport/common/krutils.cpp 3c2f037 
  libs/koreport/wrtembed/KoReportDesigner.cpp 78dda97 
  libs/koreport/wrtembed/KoReportDesignerItemBase.cpp 33bd17d 
  libs/koreport/wrtembed/KoReportDesignerItemLine.cpp e173787 
  libs/koreport/wrtembed/reportsection.cpp 2fae3d9 

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


Testing
-------


Thanks,

Dag Andersen

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


More information about the calligra-devel mailing list