<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/104626/">http://git.reviewboard.kde.org/r/104626/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On April 21st, 2012, 10:27 a.m., <b>C. Boemann</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<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 was the bug you branch will (also) fix: https://bugs.kde.org/show_bug.cgi?id=298510
</pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Hm, sure that bug is fixed by this branch? While for new documents I can confirm that the unit setting is the one of the last instance of Calligra Words that was closed, for a saved document the unit is always reset to the default, Point. Looking at the code this seems to be a mismatch of loading and storing in the Words code.
See https://git.reviewboard.kde.org/r/104701
So commiting this branch as single commit without closing that bug.</pre>
<br />
<p>- Friedrich W. H.</p>
<br />
<p>On April 16th, 2012, 9:27 p.m., Friedrich W. H. Kossebau wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Calligra and C. Boemann.</div>
<div>By Friedrich W. H. Kossebau.</div>
<p style="color: grey;"><i>Updated April 16, 2012, 9:27 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;">Sigh... so the proposal from https://git.reviewboard.kde.org/r/104607/ from boemann
"I'd say fix the code in KoUnit so it reports the unit in the order you like, and use KoUnit everywhere to define the order"
turned into quite some hacking and in the end I found myself redoing the KoUnit API partially (because the old confused me too often).
So let's have some feedback on the current state, to see what is welcome and what is not, and what else could/should be done :)
Changed:
* rename KoUnit::unit(...) to KoUnit::fromSymbol(...) <- more Qt'ish
* turn static KoUnit::unitName(KoUnit) into member method KoUnit::symbol() <- as typical use-case is on existing KoUnit instance, also shorter and more OOed
* rename KoUnit::Unit to KoUnit::Type <- "type" feels a better term here
* added KoUnit::type() and KoUnit::setFactor(...) <- useful in a few places
* remove KoUnit::unitDescription(...) from API <- not used outside
* rename KoUnit::PixelVisibility to KoUnit::ListFilter <- more general, some might want to add other flags like HideNoneMetrics
Fixes:
* ensure the same order of unit types in all unit type selectors in the UI
* update the page layout dialog on a change of the document's unit property
* update the changeUnitActions on a change of the document's unit property
</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;">Played with (hopefully) all touched widgets, seems to work.</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/karbon/image/ImageExportOptionsWidget.cpp <span style="color: grey">(2b5d541)</span></li>
<li>karbon/ui/KarbonPart.cpp <span style="color: grey">(5e6a958)</span></li>
<li>krita/plugins/extensions/imagesize/dlg_imagesize.cc <span style="color: grey">(96490c9)</span></li>
<li>krita/plugins/tools/defaulttools/kis_tool_measure.cc <span style="color: grey">(5e9afa3)</span></li>
<li>krita/ui/widgets/kis_custom_image_widget.cc <span style="color: grey">(5b00fb9)</span></li>
<li>libs/kopageapp/KoPADocument.cpp <span style="color: grey">(240171a)</span></li>
<li>libs/kopageapp/dialogs/KoPAConfigureDialog.cpp <span style="color: grey">(8912db3)</span></li>
<li>libs/koproperty/editors/spinbox.cpp <span style="color: grey">(d923c6e)</span></li>
<li>libs/koreport/common/KoReportItemBase.cpp <span style="color: grey">(7f6a575)</span></li>
<li>libs/koreport/common/krsectiondata.cpp <span style="color: grey">(38c14c8)</span></li>
<li>libs/koreport/wrtembed/KoReportDesigner.cpp <span style="color: grey">(68a61f9)</span></li>
<li>libs/koreport/wrtembed/KoReportDesignerItemBase.cpp <span style="color: grey">(f3ff8dc)</span></li>
<li>libs/koreport/wrtembed/KoReportDesignerItemLine.cpp <span style="color: grey">(ce54e7c)</span></li>
<li>libs/koreport/wrtembed/reportscene.cpp <span style="color: grey">(dd32f7a)</span></li>
<li>libs/koreport/wrtembed/reportsection.cpp <span style="color: grey">(52447cf)</span></li>
<li>libs/main/KoDocument.h <span style="color: grey">(66bf3ac)</span></li>
<li>libs/main/KoDocument.cpp <span style="color: grey">(831ed9d)</span></li>
<li>libs/main/KoRuler.cpp <span style="color: grey">(4272b9b)</span></li>
<li>libs/main/KoView.cpp <span style="color: grey">(81dafd3)</span></li>
<li>libs/main/KoView_p.h <span style="color: grey">(9b3dff2)</span></li>
<li>libs/main/config/KoConfigGridPage.h <span style="color: grey">(01373e9)</span></li>
<li>libs/main/config/KoConfigGridPage.cpp <span style="color: grey">(a3e1d6f)</span></li>
<li>libs/main/config/KoConfigMiscPage.h <span style="color: grey">(2c36996)</span></li>
<li>libs/main/config/KoConfigMiscPage.cpp <span style="color: grey">(7f54ef1)</span></li>
<li>libs/odf/KoUnit.h <span style="color: grey">(1f035fe)</span></li>
<li>libs/odf/KoUnit.cpp <span style="color: grey">(43cc908)</span></li>
<li>libs/widgets/KoPageLayoutWidget.cpp <span style="color: grey">(c9f0fc0)</span></li>
<li>libs/widgets/KoUnitDoubleSpinBox.cpp <span style="color: grey">(f9f00da)</span></li>
<li>plugins/paragraphtool/Ruler.cpp <span style="color: grey">(3053696)</span></li>
<li>plugins/textshape/dialogs/ParagraphBulletsNumbers.cpp <span style="color: grey">(87c1b85)</span></li>
<li>sheets/DocBase.cpp <span style="color: grey">(a9812c6)</span></li>
<li>sheets/dialogs/LayoutDialog.cpp <span style="color: grey">(d1090f4)</span></li>
<li>sheets/part/Doc.cpp <span style="color: grey">(4c40b87)</span></li>
<li>sheets/part/HeaderItems.cpp <span style="color: grey">(5fc3cfa)</span></li>
<li>sheets/part/HeaderWidgets.cpp <span style="color: grey">(3fac4cc)</span></li>
<li>sheets/part/dialogs/PreferenceDialog.cpp <span style="color: grey">(fae954a)</span></li>
<li>words/part/KWApplicationConfig.cpp <span style="color: grey">(b5fd980)</span></li>
<li>words/part/KWOdfLoader.cpp <span style="color: grey">(238c7fe)</span></li>
<li>words/part/dialogs/KWPageSettingsDialog.h <span style="color: grey">(0a028a9)</span></li>
<li>words/part/dialogs/KWPageSettingsDialog.cpp <span style="color: grey">(f577353)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/104626/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>