[Kde-games-devel] Review Request 115365: New design for KReversi

[Denis Kuplyakov]

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

(Updated Jan. 29, 2014, 11:13 a.m.)


Review request for KDE Games and Inge Wallin.


Changes
-------

Fixed Inge's issues.


Repository: kreversi


Description
-------

New design for KReversi with many new features and huge amount of rewrited code.

Purpose of the changes described here: http://kreversiqml.blogspot.ru/2013/08/new-kreversi-design.html
Complete scheme of new code: https://projects.kde.org/projects/kde/kdegames/kreversi/repository/entry/DESIGN_SCHEME.png?rev=deniskup%2Fnewdesign%2Fnopatch

Major features:
1) code rewrite
2) brand-new Start Game dialog
3) ability to play whites, also you can now play with a friend
4) demo mode is now a computer-computer match, and computers can have different difficulties
5...) maybe I've forgotten something


Diffs (updated)
-----

  AUTHORS 65bb7f6 
  CMakeLists.txt d0ddeea 
  DESIGN b1ae729 
  DESIGN_SCHEME.png PRE-CREATION 
  Engine.h 6f8340d 
  Engine.cpp 846b519 
  TODO d4c6bc9 
  colorscheme.h 2e5a0e7 
  colorscheme.cpp e10228c 
  commondefs.h 57b0f6a 
  commondefs.cpp PRE-CREATION 
  gamestartinformation.h PRE-CREATION 
  highscores.h aa04604 
  highscores.cpp 7c7cb56 
  kreversicomputerplayer.h PRE-CREATION 
  kreversicomputerplayer.cpp PRE-CREATION 
  kreversigame.h c54ef05 
  kreversigame.cpp f77c396 
  kreversihumanplayer.h PRE-CREATION 
  kreversihumanplayer.cpp PRE-CREATION 
  kreversiplayer.h PRE-CREATION 
  kreversiplayer.cpp PRE-CREATION 
  kreversiview.h 5529f8d 
  kreversiview.cpp 21bbe30 
  main.cpp ee8b9c7 
  mainwindow.h 3ffb2dc 
  mainwindow.cpp 0a0f5a0 
  qml/Board.qml 6657fe1 
  qml/CanvasItem.qml 9364e21 
  qml/Cell.qml e20f336 
  qml/Chip.qml a0c9d0c 
  qml/Popup.qml 378b57c 
  qml/Table.qml b989d57 
  qml/globals.js 7deec4f 
  startgamedialog.h PRE-CREATION 
  startgamedialog.cpp PRE-CREATION 
  startgamedialog.ui PRE-CREATION 

Diff: https://git.reviewboard.kde.org/r/115365/diff/


Testing
-------

Have played it a lot, all is OK!


Thanks,

Denis Kuplyakov

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-games-devel/attachments/20140129/65ec2da0/attachment-0001.html>