[Kde-games-devel] Review Request: Roll all dice when none is selected.

Luiz Romário Santana Rios luizromario at gmail.com
Thu Sep 16 17:45:08 CEST 2010


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

(Updated 2010-09-16 15:45:07.304226)


Review request for KDE Games.


Changes
-------

 - Applied requested changes
 - Fixed bug that made user be able to roll dice how many times they wanted.


Summary
-------

This patch makes "Roll" button be labeled as "Roll all" and roll all dice in the lateral widget when no die is selected.


Diffs (updated)
-----

  /trunk/KDE/kdegames/kiriki/src/diceswidget.h 1175057 
  /trunk/KDE/kdegames/kiriki/src/diceswidget.cpp 1175057 
  /trunk/KDE/kdegames/kiriki/src/lateralwidget.h 1175057 
  /trunk/KDE/kdegames/kiriki/src/lateralwidget.cpp 1175057 

Diff: http://svn.reviewboard.kde.org/r/4882/diff


Testing
-------

Works fine here.


Thanks,

Luiz Romário

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/kde-games-devel/attachments/20100916/66b4a9f7/attachment.htm 


More information about the kde-games-devel mailing list