[Kde-games-devel] Review Request: Remember|Forget instead of OK|Cancel in the highscore dialog.
Albert Astals Cid
aacid at kde.org
Wed Jul 22 22:54:29 CEST 2009
> On 2009-07-15 07:01:02, Anne-Marie Mahfouf wrote:
> > Maybe this should be submitted to the Usability Reviewboard group.
>
> Albert Astals Cid wrote:
> Makes no sense sending it to the usability mailing list while the code is wrong, Jaime are you going to answer to my comment or what?
>
> Jaime Torres wrote:
> Albert, just one line before the diffs shown in the reviewboard there is already an:
> if (askName)
> {
> ... set button names.
> }
Lol, right, now Parker can you tell us why you think it does not work at?
- Albert
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/947/#review1611
-----------------------------------------------------------
On 2009-07-15 07:23:47, Jaime Torres wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/947/
> -----------------------------------------------------------
>
> (Updated 2009-07-15 07:23:47)
>
>
> Review request for KDE Games and usability.
>
>
> 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
>
>
More information about the kde-games-devel
mailing list