[kde-edu]: Review Request: [Parley-branch] Display as many choices as set in the config options of multiple-choices mode

Daniel Laidig d.laidig at gmx.de
Mon Feb 22 23:12:50 CET 2010


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

Ship it!


You should delete the old widgets before creating new ones. I think adding  "qDeleteAll(m_choiceButtons);" before "m_choiceButtons = QList<QRadioButton*>();" should do the trick.

Currently it doesn't matter because the number doesn't change and the widgets are only created once.

- Daniel


On 2010-02-21 18:17:47, Nicolas Réau wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/2852/
> -----------------------------------------------------------
> 
> (Updated 2010-02-21 18:17:47)
> 
> 
> 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/multiplechoicemodewidget.h 1093868 
>   branches/work/parley-practice/parley/src/practice/multiplechoicemodewidget.cpp 1093868 
>   branches/work/parley-practice/parley/src/practice/practice_widget_multiplechoice.ui 1093868 
> 
> 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