<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="https://git.reviewboard.kde.org/r/118826/">https://git.reviewboard.kde.org/r/118826/</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;">This review has been submitted with commit 9e77223e000e35df08336759185d8996c0e2417f by Inge Wallin to branch master.</pre>
 <br />









<p>- Commit Hook</p>


<br />
<p>On June 23rd, 2014, 12:09 p.m. UTC, Andreas Xavier wrote:</p>








<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://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 Edu.</div>
<div>By Andreas Xavier.</div>


<p style="color: grey;"><i>Updated June 23, 2014, 12:09 p.m.</i></p>









<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;">Added statistics to report on both languages when training two way, to get the feature in before the freeze.

1. Added enum to define the statistics column names.
2. Added switch statements on practiceDirection to determine the column to display and the language to display in it.

New Problems: 
1. Changing the practice Direction doesn't update the display until you leave the screen and then return.
2. The header is only expecting a single column of day so the legend only spans the last column.</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;">Played multiplechoice forwards, backwards and two way.
</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/vocabulary/containermodel.h <span style="color: grey">(32771b5)</span></li>

 <li>src/statistics/statisticsmodel.cpp <span style="color: grey">(e195778)</span></li>

 <li>src/statistics/statisticsmainwindow.cpp <span style="color: grey">(ccf7f0a)</span></li>

 <li>src/statistics/statisticsmainwindow.h <span style="color: grey">(fe8872c)</span></li>

 <li>src/statistics/lessonstatistics.cpp <span style="color: grey">(756d96a)</span></li>

 <li>src/vocabulary/containermodel.cpp <span style="color: grey">(d8f0b00)</span></li>

</ul>

<p><a href="https://git.reviewboard.kde.org/r/118826/diff/" style="margin-left: 3em;">View Diff</a></p>







  </td>
 </tr>
</table>








  </div>
 </body>
</html>