This PR has a variety of fixes and changes:
1 - Cleaned up the win template to be more interesting. It shows a slightly different format for 2 vs 3 vs 4 players. Shows victory point cards and cards for longest road and largest army. (Issue #212)
2 - Corrected a couple of console errors related to the spinner/check when another player is finished with their round
3 - Added a trophy image for the win popup
4 - Cancel trade option resets when turn is finished (Issue #327)
5 - Restored previous version of dice roll
6 - Added robber frequency property to game.js. It controls the # of 7's allowed per set of rolls. (i.e. There are normally 6 robbers per 36 rolls. Setting "robber_frequency" to 4 would reduce that down to 4 per 36)
7 - Trade menu now closes when chat is used (Issue #326)
8 - Fixed bug allowing 2 cities to be placed in 1 spot.
flukejones
commented
7 years ago
This PR has a variety of fixes and changes: 1 - Cleaned up the win template to be more interesting. It shows a slightly different format for 2 vs 3 vs 4 players. Shows victory point cards and cards for longest road and largest army. (Issue #212) 2 - Corrected a couple of console errors related to the spinner/check when another player is finished with their round 3 - Added a trophy image for the win popup 4 - Cancel trade option resets when turn is finished (Issue #327) 5 - Restored previous version of dice roll 6 - Added robber frequency property to game.js. It controls the # of 7's allowed per set of rolls. (i.e. There are normally 6 robbers per 36 rolls. Setting "robber_frequency" to 4 would reduce that down to 4 per 36) 7 - Trade menu now closes when chat is used (Issue #326) 8 - Fixed bug allowing 2 cities to be placed in 1 spot.