<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Jul 8, 2014 at 12:32 AM, Vincent PINON <span dir="ltr"><<a href="mailto:vincent.pinon@laposte.net" target="_blank">vincent.pinon@laposte.net</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div bgcolor="#FFFFFF" text="#000000">
Hello,<br>
<br>
Sorry for the late reaction;<br>
Patch applied to activate vid.stab, on git master (stable) branch.<br>
<br>
Please test and comment ; do we need to keep older stab filters ?
(for projects compatibility)<br>
<br></div></blockquote><div><br></div><div>I recommend to remove them. Removing them does not break older projects so long as the MLT filters are still around. Keeping them only causes confusion and causes people to use the deprecated ones up until their time of removal. One can argue that "deprecated" is meant for developers, not users.</div>
<div><br></div><div>If the Kdenlive UI provides a way to re-enter the stabilize options of a stabilized clip, then perhaps you can make references to the old stabilizers open the new UI.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div bgcolor="#FFFFFF" text="#000000">
BR,<br>
<br>
Vincent<br>
<div><br>
<br>
-------- Message original --------
<table border="0" cellpadding="0" cellspacing="0">
<tbody>
<tr>
<th align="RIGHT" nowrap valign="BASELINE">Sujet: </th>
<td>Review Request 119095: GUI Added the vid.stab video
stabilizer as a menu option.</td>
</tr>
<tr>
<th align="RIGHT" nowrap valign="BASELINE">Date : </th>
<td>Thu, 03 Jul 2014 21:48:13 -0000</td>
</tr>
<tr>
<th align="RIGHT" nowrap valign="BASELINE">De : </th>
<td>Steve Guilford <a href="mailto:s.guilford@dbplugins.com" target="_blank"><s.guilford@dbplugins.com></a></td>
</tr>
<tr>
<th align="RIGHT" nowrap valign="BASELINE">Répondre
à : </th>
<td>Steve Guilford <a href="mailto:s.guilford@dbplugins.com" target="_blank"><s.guilford@dbplugins.com></a></td>
</tr>
<tr>
<th align="RIGHT" nowrap valign="BASELINE">Pour : </th>
<td>Vincent Pinon <a href="mailto:vincent.pinon@laposte.net" target="_blank"><vincent.pinon@laposte.net></a></td>
</tr>
<tr>
<th align="RIGHT" nowrap valign="BASELINE">Copie
à : </th>
<td>Steve Guilford <a href="mailto:s.guilford@dbplugins.com" target="_blank"><s.guilford@dbplugins.com></a></td>
</tr>
</tbody>
</table>
<br>
<br>
<div style="font-family:Verdana,Arial,Helvetica,Sans-Serif">
<table style="border:1px #c9c399 solid" bgcolor="#f9f3c9" cellpadding="8" width="100%">
<tbody>
<tr>
<td> This is an automatically generated e-mail. To reply,
visit: <a href="https://git.reviewboard.kde.org/r/119095/" target="_blank">https://git.reviewboard.kde.org/r/119095/</a>
</td>
</tr>
</tbody>
</table>
<br>
<table style="background-image:url('https://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png');background-repeat:repeat-x;border:1px black solid" bgcolor="#fefadf" cellpadding="8" cellspacing="0" width="100%">
<tbody>
<tr>
<td>
<div>Review request for Vincent Pinon.</div>
<div>By Steve Guilford.</div>
<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 style="border:1px solid #b8b5a0" bgcolor="#ffffff" cellpadding="10" cellspacing="0" width="100%">
<tbody>
<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">GUI Added menu option for new video stabilizer "vid.stab". Duplicated screen for videostab2 stabilizer for use with vid.stab.</pre>
</td>
</tr>
</tbody>
</table>
<h1 style="color:#575012;font-size:10pt;margin-top:1.5em">Testing </h1>
<table style="border:1px solid #b8b5a0" bgcolor="#ffffff" cellpadding="10" cellspacing="0" width="100%">
<tbody>
<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">Tested w/ current dev branch of Kdenlive.</pre>
</td>
</tr>
</tbody>
</table>
<h1 style="color:#575012;font-size:10pt;margin-top:1.5em">Diffs </h1>
<ul style="margin-left:3em;padding-left:0">
<li>src/project/clipstabilize.cpp <span style="color:grey">(3085f31)</span></li>
<li>src/mainwindow.cpp <span style="color:grey">(ba54164)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/119095/diff/" style="margin-left:3em" target="_blank">View Diff</a></p>
</td>
</tr>
</tbody>
</table>
</div>
<br>
</div>
<br>
</div>
<br>------------------------------------------------------------------------------<br>
Open source business process management suite built on Java and Eclipse<br>
Turn processes into business applications with Bonita BPM Community Edition<br>
Quickly connect people, data, and systems into organized workflows<br>
Winner of BOSSIE, CODIE, OW2 and Gartner awards<br>
<a href="http://p.sf.net/sfu/Bonitasoft" target="_blank">http://p.sf.net/sfu/Bonitasoft</a><br>_______________________________________________<br>
Kdenlive-devel mailing list<br>
<a href="mailto:Kdenlive-devel@lists.sourceforge.net">Kdenlive-devel@lists.sourceforge.net</a><br>
<a href="https://lists.sourceforge.net/lists/listinfo/kdenlive-devel" target="_blank">https://lists.sourceforge.net/lists/listinfo/kdenlive-devel</a><br>
<br></blockquote></div><br><br></div></div>