<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/128929/">https://git.reviewboard.kde.org/r/128929/</a>
</td>
</tr>
</table>
<br />
<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 The DiveO.</div>
<p style="color: grey;"><i>Updated Sept. 18, 2016, 1:18 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 patch adds the ability to not only extract a frame (in the monitors), but to immediately add it to the project for further use.
- For the UI design, this is a separate function, not folded into the existing "Save Image" dialog. Instead, the user now has an additional menu item "Extract frame to project...".
- When using "Extract frame to project", the following "Save Image to Project" file saver dialog proposes a (hopefully) suitable image file name. It bases on either the clip name (when invoked from the clip monitor) or the project name (when invoked from the project monitor), and the number of the frame extracted. This allows for a smooth workflow when extracting multiple still images from the same video clip.
- The two menu items for extracting images are corrected to end in "...", as with both menu items the user will see a dialog following menu item selection.
- The patch avoids duplicating code, so slotExtractCurrentFrame is used for both extraction, and the optional add-to-project step. This is achieved using an optional flag. The new menu item is wired up to slotExtractCurrentFrameToProject, which simply reroutes to slotExtractCurrentFrame using the optional flag.</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;">Testing on recent Kdenlive git master.</p></pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> (updated)</h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>src/monitor/monitor.h <span style="color: grey">(39679d8)</span></li>
<li>src/monitor/monitor.cpp <span style="color: grey">(8d77d8b)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/128929/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>