[prev in list] [next in list] [prev in thread] [next in thread]
List: koffice-devel
Subject: Re: Review Request: Add struct KoFontFace
From: "Thorsten Zachmann" <t.zachmann () zagge ! de>
Date: 2009-12-02 3:49:29
Message-ID: 20091202034929.3986.43053 () localhost
[Download RAW message or body]
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/2257/#review3359
-----------------------------------------------------------
trunk/koffice/libs/odf/KoFontFace.cpp
<http://reviewboard.kde.org/r/2257/#comment2697>
Please make the init list consistent with the above one.
trunk/koffice/libs/odf/KoGenStyles.cpp
<http://reviewboard.kde.org/r/2257/#comment2698>
Should we add a if in case the fontface is null, so that it is not inserted into \
the list?
- Thorsten
On 2009-12-01 11:19:59, Inge Wallin wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/2257/
> -----------------------------------------------------------
>
> (Updated 2009-12-01 11:19:59)
>
>
> Review request for KOffice.
>
>
> Summary
> -------
>
> Here is the first patch in a small patchset that will add filters for the MS 2007 \
> XML file formats. Note that I am only the middle man here.
> This patch adds the struct KoFontFace that is a public data structure representing \
> a font style. It is saved as a style:font-face element. Before this patch, only the \
> font name was available, which was saved as a style:name attribute of \
> style:font-face. Also, svg:font-family was saved as a copy.
> Now the old behaviour is still available but in addition, new attributes can be \
> specified in order to make the description of a font more accurate \
> (style:font-family-generic, svg:font-style, style:font-pitch). These are needed for \
> increasing quality of import/export filters and is already used in the MSOOXML \
> filters. The ODF specification offers even more attributes.
> The patch also contains fix for use of m_mainStyles->addFontFace() in two places, \
> both in KWord.
>
> Diffs
> -----
>
> trunk/koffice/filters/kword/msword-odf/document.cpp 1055949
> trunk/koffice/filters/kword/msword-odf/texthandler.cpp 1055949
> trunk/koffice/libs/odf/CMakeLists.txt 1055949
> trunk/koffice/libs/odf/KoFontFace.h PRE-CREATION
> trunk/koffice/libs/odf/KoFontFace.cpp PRE-CREATION
> trunk/koffice/libs/odf/KoGenStyles.h 1055949
> trunk/koffice/libs/odf/KoGenStyles.cpp 1055949
>
> Diff: http://reviewboard.kde.org/r/2257/diff
>
>
> Testing
> -------
>
>
> Thanks,
>
> Inge
>
>
_______________________________________________
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