[Kde-games-devel] Review Request: kmahjongg: Undo after won is allowed

[Albert Astals Cid]

> On Dec. 14, 2011, 1:36 p.m., Frederik Schwarzer wrote:
> > trunk/KDE/kdegames/kmahjongg/boardwidget.cpp, line 404
> > <http://svn.reviewboard.kde.org/r/6841/diff/2/?file=47251#file47251line404>
> >
> >     Hmm, I am not very happy with the message. What about: "You have finished this board, there is nothing to be undone"?
> >     
> >     Maybe a native speaker could give advice here.
> 
> Ian Wadham wrote:
>     How about "Cannot undo: you have finished this puzzle."?
>     
>     Both of the previous suggestions would be good English, but "No undo allowed" is a little impolite and "game" gets us into that "What is a game?" argument. Is it just the puzzle you are on or is it the whole KMahjongg session? English tends to be ambiguous.
> 
> Christian Krippendorf wrote:
>     I don't think so realted to "game". In KMahjongg i can choose a new game on "Game" -> "New" und also a special "Game number" so i think that "game" isn't that bad. I would prefer "game" rather than "puzzle".
>     
>     "Cannot undo: you have finished this game." <- That sounds good to me.
> 
> Ian Wadham wrote:
>     OK. Coder decides.
> 
> Christian Krippendorf wrote:
>     Finally code ok for commiting?

Wondering if it would be better just disabling the undo action?


- Albert


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


On Dec. 28, 2011, 6:56 p.m., Christian Krippendorf wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://svn.reviewboard.kde.org/r/6841/
> -----------------------------------------------------------
> 
> (Updated Dec. 28, 2011, 6:56 p.m.)
> 
> 
> Review request for KDE Games.
> 
> 
> Description
> -------
> 
> Bug List Link: https://bugs.kde.org/show_bug.cgi?id=215352
> 
> 
> Diffs
> -----
> 
>   trunk/KDE/kdegames/kmahjongg/boardwidget.cpp 1270752 
>   trunk/KDE/kdegames/kmahjongg/kmahjongg.cpp 1270752 
> 
> Diff: http://svn.reviewboard.kde.org/r/6841/diff/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Christian Krippendorf
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-games-devel/attachments/20120104/7adc46d5/attachment.html>