<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/109527/">http://git.reviewboard.kde.org/r/109527/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On June 3rd, 2013, 9:59 a.m. UTC, <b>David Faure</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;">For the record, this is waiting for an updated version of https://codereview.qt-project.org/#change,51166</pre>
</blockquote>
<p>On June 11th, 2013, 9:27 a.m. UTC, <b>Kevin Ottens</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;">Which is still not seeing any activity... Alex do you need help there?</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;">That patch really needs to send the widget to the stylehint query (animating some inheriting 3rd party widget might be broken or dog slow and then you end up with either no animations at all or for that application or crashing it) and it actually could make use of the QStyleHintReturn to determine animation duration (to be controlled by styles or a kde global setting etc.)</pre>
<br />
<p>- Thomas</p>
<br />
<p>On March 16th, 2013, 7:16 p.m. UTC, Àlex Fiestas 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 KDE Frameworks.</div>
<div>By Àlex Fiestas.</div>
<p style="color: grey;"><i>Updated March 16, 2013, 7:16 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;">Port to the incoming (hopefully) styleHint SH_Widget_Animate</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>kdeui/widgets/klineedit.cpp <span style="color: grey">(d29296c)</span></li>
<li>kdeui/widgets/klineedit_p.h <span style="color: grey">(7c58f2d)</span></li>
<li>kdeui/widgets/kmainwindow.cpp <span style="color: grey">(e629454)</span></li>
<li>kdeui/widgets/kmessagewidget.cpp <span style="color: grey">(ce0852f)</span></li>
<li>kio/kfile/kimagefilepreview.cpp <span style="color: grey">(cbd3c43)</span></li>
<li>kio/kio/kfileitemdelegate.cpp <span style="color: grey">(c11d6b6)</span></li>
<li>staging/kde4support/src/kdeui/kfadewidgeteffect.cpp <span style="color: grey">(e182a1b)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/109527/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>