[kde-edu]: Review Request: Display as many choices as set in the config options of multiple-choices mode
Nicolas Réau
thelwyn at gmail.com
Sun Feb 7 16:56:52 CET 2010
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/2852/
-----------------------------------------------------------
Review request for KDE-Edu and Frederik Gladhorn.
Summary
-------
The radio buttons in multiple choice mode are now displayed according to the option set by the user in the config dialog (instead of being hardcoded to 5 choices).
Diffs
-----
branches/work/parley-practice/parley/src/practice/abstractwidget.h 1085335
branches/work/parley-practice/parley/src/practice/flashcardmodewidget.h 1085335
branches/work/parley-practice/parley/src/practice/flashcardmodewidget.cpp 1085335
branches/work/parley-practice/parley/src/practice/guifrontend.h 1085335
branches/work/parley-practice/parley/src/practice/guifrontend.cpp 1085335
branches/work/parley-practice/parley/src/practice/multiplechoicebackendmode.cpp 1085335
branches/work/parley-practice/parley/src/practice/multiplechoicemodewidget.h 1085335
branches/work/parley-practice/parley/src/practice/multiplechoicemodewidget.cpp 1085335
branches/work/parley-practice/parley/src/practice/abstractfrontend.h 1085335
branches/work/parley-practice/parley/src/practice/practice_widget_multiplechoice.ui 1085335
branches/work/parley-practice/parley/src/practice/writtenpracticewidget.h 1085335
branches/work/parley-practice/parley/src/practice/writtenpracticewidget.cpp 1085335
Diff: http://reviewboard.kde.org/r/2852/diff
Testing
-------
It works fine here. However I'm not satisfied of my code: I feel like I'm touching some files i shouldn't have too.
Please review it and tell if it's bad implemented or how it can be improved. (btw I've not removed my debug messages from the diff yet).
Thanks,
Nicolas
More information about the kde-edu
mailing list