<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/128078/">https://git.reviewboard.kde.org/r/128078/</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 KDE Edu.</div>
<div>By Hartmut Riesenbeck.</div>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
parley
</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;">New check box in StatistcsMainWindow is added. If check box is
switched on the practice direction for the current practice mode is
saved and will be restored when this practice mode is selected again.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">This is a feature I personally missed. May it is usefull for the public.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">But I am not shure if the location for the check box, which is switching
the feature on and off, is well choosen here in the main window (see
screenshot). Or if it's better placed in the practice configure dialog.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">While working on this feature a smal bug was found in configuration
file saving. File was only saved when document was modified on
closing the application. Fixed it.</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/parleydocument.cpp <span style="color: grey">(2f5d3b32ba4462d6b9c917b109e0787cede59513)</span></li>
<li>src/parleymainwindow.cpp <span style="color: grey">(9fd1b92e88ed2614f4f71579ed0f2a31fca027c3)</span></li>
<li>src/settings/parley.kcfg <span style="color: grey">(7eac0d27a6d2f5dcb94d1ed5802b7c05b0ce4eb4)</span></li>
<li>src/statistics/statisticsmainwindow.h <span style="color: grey">(f6c87231010f672c0e32aae34345785d16484d75)</span></li>
<li>src/statistics/statisticsmainwindow.cpp <span style="color: grey">(64c7ad64057e74da5ca81b098c0e865610900bb5)</span></li>
<li>src/statistics/statisticsmainwindow.ui <span style="color: grey">(190017cc3ca788248aa468cc90349dde873fa80d)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/128078/diff/" style="margin-left: 3em;">View Diff</a></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">File Attachments </h1>
<li><a href="https://git.reviewboard.kde.org/media/uploaded/files/2016/06/01/0d19592c-0402-4b94-90e9-c5aa52fe54b6__screenshot01.png">screenshot01</a></li>
</ul>
</td>
</tr>
</table>
</div>
</body>
</html>