brianfertig
/
fertig-classic-games
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix: exclude Mastermind and Battleship from playfield selection 

Prevents the playfield option section from appearing for Mastermind and Battleship games, which do not utilize playfield tiles in the same way as other games.
This commit is contained in:
Brian Fertig 2026-05-31 12:52:16 -06:00
parent 0bb32359b2
commit 81d3a28195
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
public/src/scenes/OpponentSelectScene.js
View File

@ -102,7 +102,7 @@ export default class OpponentSelectScene extends Phaser.Scene {
[opponents[i], opponents[j]] = [opponents[j], opponents[i]];
}
const hasOptions = isCatan || isGoFish || this.gameDef.slug === 'dominion'
|| (!isWordGame && (this.cache.json.get('playfields')?.playfields?.length > 0))
|| (!isWordGame && this.gameDef.slug !== 'battleship' && this.gameDef.slug !== 'mastermind' && (this.cache.json.get('playfields')?.playfields?.length > 0))
|| this.gameDef.cardGame;
const oppScrollH = hasOptions ? OPP_SCROLL_H : 700;
this.buildOpponentGrid(opponents, oppScrollH);
@ -131,7 +131,7 @@ export default class OpponentSelectScene extends Phaser.Scene {
if (this.gameDef.slug === 'mastermind') this.buildSecretRevealTypeSection(340, 1013);
if (!isWordGame) {
if (!isWordGame && this.gameDef.slug !== 'battleship' && this.gameDef.slug !== 'mastermind') {
this.buildOptionSection('Playfield', 630, this.cache.json.get('playfields')?.playfields ?? [],
'selectedPlayfield', 'playfieldTiles', (pf) => this.selectPlayfield(pf));
}
@ -143,7 +143,7 @@ export default class OpponentSelectScene extends Phaser.Scene {
}
// Apply playfield default; card back is chosen randomly
if (!isWordGame) {
if (!isWordGame && this.gameDef.slug !== 'battleship' && this.gameDef.slug !== 'mastermind') {
const pfd = this.cache.json.get('playfields') ?? {};
this.applyDefault('playfields', pfd.default, 'selectedPlayfield', 'playfieldTiles');
}