GNOME Bugzilla – Bug 747583
New game: Wall
Last modified: 2015-06-09 16:08:16 UTC
Created attachment 301235 [details] [review] New game Wall for Aisleriot This is a game that me and my two friends came up with. The game uses a double deck. 45 cards are dealt on the table in a wall formation. Above the wall are 'guard post' piles and a pile with two kings. The player attacks first the wall, then the guard posts and finally the kings with the remaining cards. The game is won if the player is able to beat the two kings at the top before he or she runs out of attack cards. An attack succeeds if the sum of values of the attack cards is higher than the 'health' of the cards under attack. Health is also computed as a certain sum of card values. Detailed rules are in the attached patch in help/C/wall.xml . The patch can be applied against the current head at github master branch, ebace20fa7.
Hi, thanks for the patch. Looks pretty solid overall, I just have some minor comments. The hint function will tell you whether you can attack a face-down slot and win, and the value of its top card. In a future version of Aisleriot, hints may be changed to have a visual indicator of the slots involved, and if so this would also reveal which face-down slot can be successfully attacked. I'd probably change find-winnable-slot to only account for face-up slots, and have a separate type of hint that advises attacking face-down slots unconditionally. (droppable?) should return false when start-slot and end-slot are equal, to avoid creating an empty undo entry. Do the kings have to be evil? I don't feel like it's necessary for the game's narrative to judge the morality of the player's actions. GNOME seems to recommend avoiding personal pronouns in documentation other than "you". Maybe references to "the player" in documentation should be replaced with "you"? Just mentioning this because gendered language is something I've become sensitive to recently, not trying to nit-pick your writing style. I wouldn't be surprised if we do the same thing in existing documentation, including things I've written, as I didn't really think about it until now. https://developer.gnome.org/gdp-style-guide/2.32/gdp-style-guide.html#locale-5
Created attachment 302332 [details] [review] New game Wall for Aisleriot - Updated
Thanks a lot for the corrections and suggestions. I think they were all good and I fixed the problems as you suggested. I also made another minor change: even if you can't remove any visible cards (so you also can't win), the game won't end if you can still attack a face-down slot. This makes sense since you can potentially increase the score by attacking that slot, and furthermore, in every other situation as well the game continues until there are no possible moves. I attached an updated patch.
Looks good to me.
Could someone commit this for me? I don't have commit access.
Pushed to master.