Re: [Kde-games-devel] Review Request: Remember|Forget instead of OK|Cancel in the highscore dialog.

> On 2009-07-07 09:15:16, Parker Coates wrote:
> > Interesting idea, but as KScoreDialog is currently implemented, clicking 
> > the Cancel button doesn't actually forget the score. It just cancels your 
> > opportunity to change the name associated with the score.

I've checked again the code (and played a lot to kollision) to check it. 
And as I always get the name prompt, when I click on the Forget button, the 
score is not saved.
You can see that the saveScores() method is only called when the Remember 
button is pressed or when the name prompt is not shown (never happened to me 
yet). 


- Jaime


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


On 2009-07-07 08:57:14, Jaime Torres wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/947/
> -----------------------------------------------------------
> 
> (Updated 2009-07-07 08:57:14)
> 
> 
> Review request for KDE Games.
> 
> 
> Summary
> -------
> 
> Use the more descriptive verbs Remember and Forget (this highscore) instead 
> of OK|Cancel.
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdegames/libkdegames/highscore/kscoredialog.cpp 953006 
> 
> Diff: http://reviewboard.kde.org/r/947/diff
> 
> 
> Testing
> -------
> 
> The dialog works as before.
> 
> 
> Thanks,
> 
> Jaime
> 
>

_______________________________________________
kde-games-devel mailing list
kde-games-devel@xxxxxxx
https://mail.kde.org/mailman/listinfo/kde-games-devel