<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 />
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">This review has been submitted with commit 9b7176353bb45354c627f2f69c3f49be28ed8834 by Friedrich W. H. Kossebau to branch master.</pre>
<br />
<p>- Commit</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>