nine_men_morris, complete the AI part
Description
Description
Details
Details
- Committed
pgupta Jun 21 2016, 6:22 PM - Parents
- R2:ea9eb3de9700: nine_men_morris, fix some bugs and issues
- Branches
- Unknown
- Tags
Comment Actions
You should not update level when in tutorial mode if you press previous/nextLevel.
| /src/activities/nine_men_morris/NineMenMorris.qml | ||
|---|---|---|
| 723 | I think this text will never be displayed so yuo can remove the line | |
| 759 | why 4? I tested with 2 and the tutorial images were bigger and didn't override with the bar | |
| /src/activities/nine_men_morris/nine_men_morris.js | ||
| 83 | 24 is hard coded, can you do a variable and use it everywhere instead? | |
| 559 | better name? | |
| 563 | you can store itemAt(i) directly there instead of repeating it | |
| 715 | you can create local variables for items.dragPoints.itemAt(index).****Point to avoid going through the repeater each time | |
| 1198 | if it is an integer, I think it's clearer to do != 0 | |