Review Request 118826: Added statistics display of second language trained in multi-way training.

Andreas Xavier andxav at zoho.com
Mon Jun 23 12:46:47 UTC 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118826/
-----------------------------------------------------------

(Updated June 23, 2014, 12:46 p.m.)


Status
------

This change has been marked as submitted.


Review request for KDE Edu.


Repository: parley


Description
-------

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.


Diffs
-----

  src/vocabulary/containermodel.h 32771b5 
  src/statistics/statisticsmodel.cpp e195778 
  src/statistics/statisticsmainwindow.cpp ccf7f0a 
  src/statistics/statisticsmainwindow.h fe8872c 
  src/statistics/lessonstatistics.cpp 756d96a 
  src/vocabulary/containermodel.cpp d8f0b00 

Diff: https://git.reviewboard.kde.org/r/118826/diff/


Testing
-------

Played multiplechoice forwards, backwards and two way.


Thanks,

Andreas Xavier

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20140623/b92f4b94/attachment-0001.html>


More information about the kde-edu mailing list