Review Request 127505: Parley: Randomize order of letters in Mixed Letters Practice

[Andreas Cord-Landwehr]

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



The change looks good to me. Also considering the commit message by Inge (btw in Sweden "Inge" is a male name AFAIK ;), this pretty much looks like a KF5 porting bug to me.
Please wait a few days if you get a reply, else consider this as a "Ship-It". We should get this patch into the coming release.

- Andreas Cord-Landwehr


On März 27, 2016, 2:13 vorm., Julian Helfferich wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/127505/
> -----------------------------------------------------------
> 
> (Updated März 27, 2016, 2:13 vorm.)
> 
> 
> Review request for KDE Edu and Inge Wallin.
> 
> 
> Bugs: 360239
>     http://bugs.kde.org/show_bug.cgi?id=360239
> 
> 
> Repository: parley
> 
> 
> Description
> -------
> 
> The order of letters in the Mixed Letter Practice is currently not randomized. Instead, the letters are only shifted vertically (see bug 360239).
> 
> Now, I use KRandomSequence::randomize() to shuffle the list of characters comprising the solution. The KRandomSequence is a member of MixedLettersModeWidget. Thus, it gets instantiated each time the Mixed Letters Practice is started. I also use KRandomSequence::getInt() instead of KRandom::random() for consistency.
> 
> I have added Inge Wallin to the reviewers, because she replaced KRandomSequence with KRandom about a year ago. Is there a good reason not to use KRandomSequence?
> 
> 
> Diffs
> -----
> 
>   src/practice/mixedlettersmodewidget.h f2acaa1 
>   src/practice/mixedlettersmodewidget.cpp adc53cd 
> 
> Diff: https://git.reviewboard.kde.org/r/127505/diff/
> 
> 
> Testing
> -------
> 
> Build and tested on Kubuntu 15.10:
> 
> Practiced vocabulary using Mixed Letter Practice and confirmed that letters are randomized.
> 
> 
> Thanks,
> 
> Julian Helfferich
> 
>

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