<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/108097/">http://git.reviewboard.kde.org/r/108097/</a>
</td>
</tr>
</table>
<br />
<p>Ship it!</p>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">okay, enough time has gone with no one complaining - let's just approve</pre>
<br />
<p>- C.</p>
<br />
<p>On January 3rd, 2013, 12:05 p.m. UTC, Friedrich W. H. Kossebau wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Calligra.</div>
<div>By Friedrich W. H. Kossebau.</div>
<p style="color: grey;"><i>Updated Jan. 3, 2013, 12:05 p.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">There are a few left-overs from <Qt4 times were it was fashion to use KDialog::spacingHint and KDialog::marginHint as well as hardcode the default spacing/margin with values that seemed sane (who imagined the code to still be in use with displays one did not even dream of).
No longer needed these days, as the styles care for proper spacings. Only exception is "margin=0", which means no margin here and is still useful and needed.
Spared out Kexi, Krita and Plan UI files in this RR, as they seem to be full of pixel-optimized layouting, will need more time for those.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Looked at all UI files with designer, looked all okay/better.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>filters/flow/imageexport/flow_imageexportwidget.ui <span style="color: grey">(05fe6f9)</span></li>
<li>filters/words/ascii/ImportDialogUI.ui <span style="color: grey">(eec1a36)</span></li>
<li>libs/main/KoDetailsPaneBase.ui <span style="color: grey">(1bb80f0)</span></li>
<li>libs/main/KoOpenPaneBase.ui <span style="color: grey">(b824720)</span></li>
<li>libs/main/config/KoConfigAuthorPage.ui <span style="color: grey">(2a3aea4)</span></li>
<li>libs/main/koDocumentInfoAboutWidget.ui <span style="color: grey">(1695ae4)</span></li>
<li>libs/main/koDocumentInfoAuthorWidget.ui <span style="color: grey">(c6a9da9)</span></li>
<li>libs/widgets/KoCsvImportDialog.ui <span style="color: grey">(645cf64)</span></li>
<li>plugins/textshape/dialogs/FontDecorations.ui <span style="color: grey">(2f9aca1)</span></li>
<li>plugins/variables/FixedDateFormat.ui <span style="color: grey">(ff181ac)</span></li>
<li>sheets/dialogs/DataFormatWidget.ui <span style="color: grey">(5205d22)</span></li>
<li>sheets/dialogs/FontWidget.ui <span style="color: grey">(d0cb129)</span></li>
<li>sheets/dialogs/PositionWidget.ui <span style="color: grey">(2b5dedb)</span></li>
<li>sheets/dialogs/ProtectionWidget.ui <span style="color: grey">(62f20d7)</span></li>
<li>sheets/dialogs/SpecialPasteWidget.ui <span style="color: grey">(623d107)</span></li>
<li>sheets/dialogs/SubtotalWidget.ui <span style="color: grey">(1a68bf7)</span></li>
<li>sheets/part/dialogs/SheetPropertiesWidget.ui <span style="color: grey">(bb73166)</span></li>
<li>sheets/part/dialogs/SheetSelectWidget.ui <span style="color: grey">(3842e7a)</span></li>
<li>sheets/plugins/scripting/scripts/htmlexportinfos.ui <span style="color: grey">(214ed86)</span></li>
<li>sheets/plugins/solver/SolverDetailsWidget.ui <span style="color: grey">(18a09d7)</span></li>
<li>sheets/plugins/solver/SolverMainWidget.ui <span style="color: grey">(d57e35b)</span></li>
<li>words/part/dialogs/KWStartupWidget.ui <span style="color: grey">(c198d88)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/108097/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>