<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 />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On September 18th, 2016, 9:57 a.m. UTC, <b>Jean-Baptiste Mardelle</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<table width="100%" border="0" bgcolor="white" style="border: 1px solid #C0C0C0; border-collapse: collapse; margin: 2px padding: 2px;">
<thead>
<tr>
<th colspan="4" bgcolor="#F0F0F0" style="border-bottom: 1px solid #C0C0C0; font-size: 9pt; padding: 4px 8px; text-align: left;">
<a href="https://git.reviewboard.kde.org/r/128929/diff/1/?file=476999#file476999line950" style="color: black; font-weight: bold; text-decoration: underline;">src/monitor/monitor.cpp</a>
<span style="font-weight: normal;">
(Diff revision 1)
</span>
</th>
</tr>
</thead>
<tbody>
<tr>
<th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2"></font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>
<th bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">950</font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> <span class="k">if</span> <span class="p">(</span><span class="n">addToProject</span> <span class="o">&&</span> <span class="n">QFileInfo</span><span class="p">(</span><span class="n">path</span><span class="p">).</span><span class="n">fileName</span><span class="p">().</span><span class="n">isEmpty</span><span class="p">())</span> <span class="p">{</span></pre></td>
</tr>
</tbody>
</table>
<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;">Minor issue, but if the project has not yet been saved, and attempting to add a frame to project from timeline, the suggested name is:
"-f320.png" since project url is empty. Might be better to preprend i18n("untitled") in case there is no project url.</p></pre>
</blockquote>
</blockquote>
<pre style="margin-left: 1em; 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;">For some reason I assumed the project URL to contain "unnamed", but did not thoroughly check. Updated patch now checks that project URL isValid() and otherwise uses i18n("untitled"). This now works as expected.</p></pre>
<br />
<p>- The</p>
<br />
<p>On September 18th, 2016, 1:18 p.m. UTC, The DiveO 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 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> </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>