Review Request: Improve the layout of KoPageLayoutWidget and KWDocumentColumns, also add margins to print option pages
Friedrich W. H. Kossebau
kossebau at kde.org
Thu Mar 22 23:42:28 GMT 2012
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/104370/
-----------------------------------------------------------
(Updated March 22, 2012, 11:42 p.m.)
Review request for Calligra, C. Boemann and Dag Andersen.
Changes
-------
Forgot to add what I like in these cases: before-after pictures.
View e.g. in Gwenview and switch fast between the images (Left- and Right keys) to see the changes better.
Description
-------
Small improvements (?) to things which jumped into my eyes a few times meanwhile.
* Changed: printing option widgets need to have outer margins set, due to being set to a tab page
* Changed: no need to set margins of KoPageLayoutWidget object to 0, is by default
* Changed: improve layout of KWDocumentColumns by using a QFormLayout
* Changed: make labels of KoPageLayoutWidget right-aligned, improve resize behaviour
Not sure if KoPageLayoutWidget should be resizable/enlargable at all, but it's consistent to other UIs,
so I have implemented some nicer enlarging behaviour there.
Okay to backport all to 2.4?
Diffs
-----
libs/main/KoMainWindow.cpp 627d763
libs/widgets/KoPageLayoutDialog.cpp 7eaeb6a
libs/widgets/KoPageLayoutWidget.ui b36e080
plan/libs/ui/kptganttprintingoptions.ui 2d9d85b
plan/libs/ui/kptviewbase.cpp 90bbad5
words/part/dialogs/KWDocumentColumns.ui 56e58ca
words/part/dialogs/KWStartupWidget.cpp a0a43ac
Diff: http://git.reviewboard.kde.org/r/104370/diff/
Testing
-------
Used all the UIs touched and resized them a few times.
Thanks,
Friedrich W. H. Kossebau
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20120322/61d04c6f/attachment.htm>
More information about the calligra-devel
mailing list