<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/107890/">http://git.reviewboard.kde.org/r/107890/</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;">I tried your code. Please look at http://wstaw.org/m/2013/01/04/plasma-desktopcp2263.png
QLocale does not seem to follow KDE Locale here. But I guess you know that, so I am OK with the patch. I propose to add a //! @todo at least in both classes.
We can say koproperty can be kdelibs-dependent for now; when some of kdelibs land in Qt 5.x we'll have opportunity for modularization.
See also: http://community.kde.org/KDE_Core/KLocale/Frameworks#Calendar_.2F_Date_.2F_Time</pre>
<br />
<p>- Jarosław</p>
<br />
<p>On December 24th, 2012, 12:04 a.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 Jarosław Staniek.</div>
<div>By Friedrich W. H. Kossebau.</div>
<p style="color: grey;"><i>Updated Dec. 24, 2012, 12:04 a.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;">Follow-up to https://git.reviewboard.kde.org/r/107792/
No time/concentration yet found to do as suggested and follow the implementation of "KexiDateTableEdit/KexiTimeTableEdit/KexiDateTimeTableEdit (with Kexi*Formatter)".
But this patch should at least improve the current situation, having the same format in display and edit mode, and also painting the upper border in edit mode.
Switched from using KDE KLocale to QLocale for the display (as that is also what is used in the QDateTimeEdit). What should be used in an implementation done as suggested? Use as much kdelibs as possible, or should the KoProperty code trying to be more Qt-only?
Patch also adds min & max options to DateTimeEdit.
</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>libs/koproperty/editors/dateedit.h <span style="color: grey">(356b236)</span></li>
<li>libs/koproperty/editors/dateedit.cpp <span style="color: grey">(3daa163)</span></li>
<li>libs/koproperty/editors/datetimeedit.h <span style="color: grey">(bb04a9c)</span></li>
<li>libs/koproperty/editors/datetimeedit.cpp <span style="color: grey">(09ca91e)</span></li>
<li>libs/koproperty/editors/timeedit.h <span style="color: grey">(6b9861e)</span></li>
<li>libs/koproperty/editors/timeedit.cpp <span style="color: grey">(9bc8e48)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/107890/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>