> On May 6, 2012, 9:10 p.m., Jarosław Staniek wrote: > > libs/koreport/renderer/odt/KoSimpleOdtCheckBox.cpp, line 60 > > <http://git.reviewboard.kde.org/r/104797/diff/1/?file=59661#file59661line60> > > > > Is't 0.0pt enough? > > Dag Andersen wrote: > I'm not sure, but: > This is the way words and LO formats pt values (probably for a reason), > so I would like keep it this way too. > (Same for the following similar cases) > > C. Boemann wrote: > uhm we do gs.addPropertyPt("fo:margin", 0); > > places where we do as you say would be maybe not bug but wrong (note the > different method)
Ahhh, so it just ends up in the xml file in the long format. Ok, I'll use the gs.addPropertyPt() instead... Done. - Dag ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104797/#review13520 ----------------------------------------------------------- On April 30, 2012, 4:31 p.m., Dag Andersen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/104797/ > ----------------------------------------------------------- > > (Updated April 30, 2012, 4:31 p.m.) > > > Review request for Calligra, Jarosław Staniek and Adam Pigg. > > > Description > ------- > > Changes is in branch: report_odt_danders > > Move plans odt export into libs/koreport. > > Retains the current odt export in koreport (as odttables) and > adds the new as odtframes. > > Patch includes modifications to Plan to enable use of both formats. > > > Diffs > ----- > > libs/koreport/CMakeLists.txt d0737c9 > libs/koreport/renderer/KoReportODTRenderer2.h PRE-CREATION > libs/koreport/renderer/KoReportODTRenderer2.cpp PRE-CREATION > libs/koreport/renderer/KoReportRendererBase.cpp 497177d > libs/koreport/renderer/odt/KoSimpleOdtCheckBox.h PRE-CREATION > libs/koreport/renderer/odt/KoSimpleOdtCheckBox.cpp PRE-CREATION > libs/koreport/renderer/odt/KoSimpleOdtDocument.h PRE-CREATION > libs/koreport/renderer/odt/KoSimpleOdtDocument.cpp PRE-CREATION > libs/koreport/renderer/odt/KoSimpleOdtImage.h PRE-CREATION > libs/koreport/renderer/odt/KoSimpleOdtImage.cpp PRE-CREATION > libs/koreport/renderer/odt/KoSimpleOdtLine.h PRE-CREATION > libs/koreport/renderer/odt/KoSimpleOdtLine.cpp PRE-CREATION > libs/koreport/renderer/odt/KoSimpleOdtPicture.h PRE-CREATION > libs/koreport/renderer/odt/KoSimpleOdtPicture.cpp PRE-CREATION > libs/koreport/renderer/odt/KoSimpleOdtPrimitive.h PRE-CREATION > libs/koreport/renderer/odt/KoSimpleOdtPrimitive.cpp PRE-CREATION > libs/koreport/renderer/odt/KoSimpleOdtTextBox.h PRE-CREATION > libs/koreport/renderer/odt/KoSimpleOdtTextBox.cpp PRE-CREATION > libs/koreport/renderer/odt/ReportODTRenderer.h PRE-CREATION > plan/libs/ui/CMakeLists.txt 4948bd3 > plan/libs/ui/reports/odt/KoSimpleOdtCheckBox.h 94123e2 > plan/libs/ui/reports/odt/KoSimpleOdtCheckBox.cpp 9a94f5b > plan/libs/ui/reports/odt/KoSimpleOdtDocument.h 65bbbee > plan/libs/ui/reports/odt/KoSimpleOdtDocument.cpp 7d917ef > plan/libs/ui/reports/odt/KoSimpleOdtImage.h 3c40922 > plan/libs/ui/reports/odt/KoSimpleOdtImage.cpp 1061664 > plan/libs/ui/reports/odt/KoSimpleOdtLine.h 6ea097b > plan/libs/ui/reports/odt/KoSimpleOdtLine.cpp 2347117 > plan/libs/ui/reports/odt/KoSimpleOdtPicture.h 10c3b85 > plan/libs/ui/reports/odt/KoSimpleOdtPicture.cpp e0df27f > plan/libs/ui/reports/odt/KoSimpleOdtPrimitive.h 7cff1c7 > plan/libs/ui/reports/odt/KoSimpleOdtPrimitive.cpp ddea498 > plan/libs/ui/reports/odt/KoSimpleOdtTextBox.h 428dff7 > plan/libs/ui/reports/odt/KoSimpleOdtTextBox.cpp d775689 > plan/libs/ui/reports/odt/ReportODTRenderer.h 3c9f149 > plan/libs/ui/reports/odt/ReportODTRenderer.cpp 2c95ed2 > plan/libs/ui/reports/report.h 8cef5be > plan/libs/ui/reports/reportexportpanel.cpp 973078d > plan/libs/ui/reports/reportview.h 265080a > plan/libs/ui/reports/reportview.cpp dea21a6 > > Diff: http://git.reviewboard.kde.org/r/104797/diff/ > > > Testing > ------- > > Tested with Plan, resulting files opened with Words. > > > Thanks, > > Dag Andersen > >
_______________________________________________ calligra-devel mailing list calligra-devel@kde.org https://mail.kde.org/mailman/listinfo/calligra-devel
