Review Request 118826: Added statistics display of second language trained in multi-way training.
Andreas Xavier
andxav at zoho.com
Mon Jun 23 12:09:34 UTC 2014
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118826/
-----------------------------------------------------------
(Updated June 23, 2014, 12:09 p.m.)
Review request for KDE Edu.
Changes
-------
Removes 2 of 3 enums.
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 (updated)
-----
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/00d236e2/attachment.html>
More information about the kde-edu
mailing list