<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="12" style="border: 1px #c9c399 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://git.reviewboard.kde.org/r/129270/">https://git.reviewboard.kde.org/r/129270/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On October 31st, 2016, 6:39 p.m. UTC, <b>Jean-Baptiste Mardelle</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;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Thanks, had a quick look and seems ok, definitely a step in the right direction.
Just to make sure you are aware of it, Kdenlive 16.12 feature freeze is on the 17th of november, after that only bug fixes should get in (A 16.12 branch will be created and master will be open for new developments).
I can merge your changes now if they are safe, but if you plan more imnportant changes that could introduce regressions, might be better to wait until the 17th when we can commit to master without consequences for the 16.12 release.
Also I think I will ask the sysadmins to give you commit access if you want, it will make it easier for you to work.</p></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;">It is always possible (and recommended) to create a new development branch to store the WIP aside from master until the task gets completed ; on KDE infra it can also be done in a personal clone...
And yes I would support the request for a KDE dev account for you.
Note: KDE doesn't allow pseudo's as committers, you will have to declare your real name...</pre>
<br />
<p>- Vincent</p>
<br />
<p>On October 31st, 2016, 5:30 p.m. UTC, alcinos alcinos wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="12" style="border: 1px #888a85 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
<div>Review request for Kdenlive.</div>
<div>By alcinos alcinos.</div>
<p style="color: grey;"><i>Updated Oct. 31, 2016, 5:30 p.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
kdenlive
</div>
<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;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">This is a first step towards a refactoring of the effects stack. Currently the different widgets do not share a base class and their creation based on the xml is a bit cluttered, with a lot of redundancies. The plan is to unify this by creating a proper factory method.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">I'll split the work into small and hopefully easy to review bits so that we make sure that no feature is lost on the way.</p></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;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Tested with a mock project</p></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>src/CMakeLists.txt <span style="color: grey">(2814fa3)</span></li>
<li>src/bin/generators/generators.cpp <span style="color: grey">(03f1479)</span></li>
<li>src/effectstack/CMakeLists.txt <span style="color: grey">(80ae0d6)</span></li>
<li>src/effectstack/collapsiblegroup.h <span style="color: grey">(d0c0f00)</span></li>
<li>src/effectstack/effectstackedit.h <span style="color: grey">(4d51f9b)</span></li>
<li>src/effectstack/geometryval.h <span style="color: grey">(82d7e71)</span></li>
<li>src/effectstack/geometryval.cpp <span style="color: grey">(3e0436c)</span></li>
<li>src/effectstack/keyframeedit.h <span style="color: grey">(f63f697)</span></li>
<li>src/effectstack/keyframeedit.cpp <span style="color: grey">(5739ea0)</span></li>
<li>src/effectstack/parametercontainer.h <span style="color: grey">(3b1b1b3)</span></li>
<li>src/effectstack/parametercontainer.cpp <span style="color: grey">(ccec798)</span></li>
<li>src/effectstack/widgets/abstractparamwidget.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>src/effectstack/widgets/animationwidget.h <span style="color: grey">(4e13503)</span></li>
<li>src/effectstack/widgets/animationwidget.cpp <span style="color: grey">(cc86e26)</span></li>
<li>src/effectstack/widgets/boolparamwidget.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>src/effectstack/widgets/boolparamwidget.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>src/effectstack/widgets/cornerswidget.h <span style="color: grey">(221bb3d)</span></li>
<li>src/effectstack/widgets/doubleparameterwidget.h <span style="color: grey">(7c81a2e)</span></li>
<li>src/effectstack/widgets/doubleparameterwidget.cpp <span style="color: grey">(bf7f245)</span></li>
<li>src/effectstack/widgets/draggablelabel.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>src/effectstack/widgets/draggablelabel.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>src/effectstack/widgets/geometrywidget.h <span style="color: grey">(681e5cb)</span></li>
<li>src/effectstack/widgets/geometrywidget.cpp <span style="color: grey">(9201e3b)</span></li>
<li>src/effectstack/widgets/listparamwidget.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>src/effectstack/widgets/listparamwidget.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>src/ui/boolval_ui.ui <span style="color: grey">(5f22525)</span></li>
<li>src/ui/geometryval_ui.ui <span style="color: grey">(1705840)</span></li>
<li>src/ui/listval_ui.ui <span style="color: grey">(7bc1925)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/129270/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>