Date: prev next · Thread: first prev next last
2012 Archives by date, by thread · List index


Jan Holesovsky wrote (20-02-12 18:04)

Nope, still 3 I am afraid :-( - after the feature freeze, before we even
needed 1 review for pushing to the libreoffice-3-5 branch, it was 3
reviews for a feature or two that got in few days after the feature
freeze (I remember the SmartArt import).


I can give some explanation on my changes (I have posted these to bug44516 itself too):
The changes have to do with the addition of page width and height to label definitions
and with the creation of a writer document with frames representing labels or business
cards.
-the majority of the changes (let's say 98%) regards the addition of members lPWidth 
and lPHeight to class SwLabItem. The label definition file (Labels.xcu) has had all
labels changed and various cxx files have the member added for assigning values to
class instances; There is one special case in labelcfg.cxx (lines 120-126) which
addresses the possibility that a non-converted (no page width/height defined) is read.
-the page width/height is also shown in the dialog, straightforward code including
setting minimum and maximum values that can be entered.
-There are cases where the summed width of the frames exceeds the page width (same for 
height). Example: an A4 sheet with 3 labels 70mm wide produces a frame width of 3969
twips (total width 3 x 3969 = 11907) whereas the page is only 11906twips wide. In
applab.cxx, iResultWidth, iResultHeight, iWidth, iHeight compensate for this.
-Finally there is the case where the horizontal gap between labels is larger than the
right margin (same for height). The document is created with frames that have right and 
lower spaces representing the horizontal/vertical gap. The last frame of the column/row 
will not fit on the row/column if the right/lower page margin is not big enough. To solve 
that, I use different frames for the right column and for the bottom row. (applab.cxx, 
pFmtOERow, pFmtEOCol, pFmtEOColEORow).

I hope this text will be of assistence to reviewers.

Winfried



Context


Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.