[prev in list] [next in list] [prev in thread] [next in thread]
List: koffice-devel
Subject: Re: Review Request: Loading and saving of custom path data for
From: "Thorsten Zachmann" <t.zachmann () zagge ! de>
Date: 2010-01-22 7:23:44
Message-ID: 20100122072344.30333.43002 () localhost
[Download RAW message or body]
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/2634/#review3788
-----------------------------------------------------------
I get an compile error. Looks like you forgot to add KoConnectorShape_p.h and some \
some other stuff in the patch.
- Thorsten
On 2010-01-20 20:51:32, Jan Hambrecht wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/2634/
> -----------------------------------------------------------
>
> (Updated 2010-01-20 20:51:32)
>
>
> Review request for KOffice.
>
>
> Summary
> -------
>
> This patch implements loading and saving of path data of a connector shape as \
> defined by ODF 1.2. Connector shapes not connected completely with both ends of the \
> connector to an existing shape can preserve their path data by saving it to a svg:d \
> attribute plus an additional svg:viewBox attribute. The exact details are properly \
> underdefined in the ODF specification, as usual. I implemented it the way which \
> made most sense to me. Note that OOo seems to not save the requiered svg:viewBox \
> attribute, but the code does deal with that case sufficiently. In order to reuse \
> some existing code from KoPathShape I moved the private KoPathShape::map function \
> to the shared Private class. Also I added a default parameter value to the \
> KoPathShape::toString function as it seems in most cases a identiy matrix was \
> passed anyway.
>
> Diffs
> -----
>
> /trunk/koffice/libs/flake/KoConnectionShape.h 1075815
> /trunk/koffice/libs/flake/KoConnectionShape.cpp 1075815
> /trunk/koffice/libs/flake/KoConnectionShapeLoadingUpdater.cpp 1075815
> /trunk/koffice/libs/flake/KoPathShape.h 1075815
> /trunk/koffice/libs/flake/KoPathShape.cpp 1075815
> /trunk/koffice/libs/flake/KoPathShape_p.h 1075815
>
> Diff: http://reviewboard.kde.org/r/2634/diff
>
>
> Testing
> -------
>
> I tested with an OOo exported odp file which contained 3 different unconnected \
> connector shapes. The additional saved attributes do validate when checking with \
> the online ODF validator (there are some errors thrown from other \
> elements/attributes in our files, which do show some errors on the validator side, \
> i.e. regarding draw:layer-set element).
>
> Thanks,
>
> Jan
>
>
_______________________________________________
koffice-devel mailing list
koffice-devel@kde.org
https://mail.kde.org/mailman/listinfo/koffice-devel
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic