[kde-edu]: Review Request: [Parley-branch] 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:57:31 CET 2010


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/2852/
-----------------------------------------------------------

(Updated 2010-02-07 15:57:30.841654)


Review request for KDE-Edu and Frederik Gladhorn.


Summary (updated)
-------

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