<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/108653/">http://git.reviewboard.kde.org/r/108653/</a>
</td>
</tr>
</table>
<br />
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I really like that approach, it's much more smart (and we love smart things) than a clumsy setting saying "Auto-resume". I am not sure whether we should expose that config option to the user (given that Dragon hasn't got any settings) but having it changeable editing the config files should work for those who want that. And not sure if 30 days isn't too long.
Also, I think we need some auto-prune feature so the config file where the last playback date is stored doesn't get full with files I once watched and deleted/forgot.</pre>
<br />
<p>- Kai Uwe</p>
<br />
<p>On January 29th, 2013, 10:28 a.m. UTC, Andrea Scarpino 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 Multimedia.</div>
<div>By Andrea Scarpino.</div>
<p style="color: grey;"><i>Updated Jan. 29, 2013, 10:28 a.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;">This patch allow dragon to not restore the cursor position in the sidebar if the video has been opened one month ago.
We could also make this time configurable in the settings (e.g. "Expires profiles after 30 days").</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;">Build, open a video, move the cursor, close dragon, change the Date line in the rc, open the video again.
The video starts from the begin.</pre>
</td>
</tr>
</table>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="http://bugs.kde.org/show_bug.cgi?id=251921">251921</a>
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>src/app/mainWindow.h <span style="color: grey">(60dfcf6)</span></li>
<li>src/app/mainWindow.cpp <span style="color: grey">(55e89df)</span></li>
<li>src/app/videoWindow.cpp <span style="color: grey">(a87715f)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/108653/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>