Confirm before New Game discards a game in progress
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 78/100
- Type d'issue
- Fonctionnalité
- Clarté
- Clairement spécifiée
- Activité
- Active
- Stack technique
- javascript
- Domaine
- frontend
Piste de recherche
Lis js/game.js:41-42 et js/game.js:76-91 pour suivre comment New Game appelle resetGame, puis examine index.html:58-61 pour repérer les commandes adjacentes. Choisis l’une des deux approches proposées et vérifie comment l’état du jeu et l’historique des coups sont stockés. Le travail est terminé lorsqu’une partie en cours ne peut pas être abandonnée d’un seul toucher, tandis que démarrer une nouvelle partie sans aucun coup ou après la fin de la partie ne nécessite aucune étape supplémentaire.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Summary
The New Game button sits next to Undo and immediately wipes the board, the move list, the undo stack and the saved game, with no confirmation and no way back.
Where
js/game.js:41-42(button wired straight toresetGame)js/game.js:76-91(resetGame)index.html:58-61(the two adjacent buttons)
Proposal
When moves have been played and the game is not over, either ask for confirmation with an inline two-step button (avoiding window.confirm), or keep the previous game in memory and show a short "Game reset. Undo" control that restores it. No prompt when the game has ended or no moves were made.
Who benefits
Anyone who mis-taps next to Undo, most likely on a phone.
Acceptance criteria
- A game in progress cannot be discarded with a single tap
- No extra step when there is nothing to lose
Getting started
- Relevant files:
js/game.js,index.html - How to run tests:
npm install, thennpm testandnpm run lint - Related docs:
docs/ARCHITECTURE.md,.github/CONTRIBUTING.md
Filed by repo-audit/full-audit at 8a2c86c. Severity: medium. Effort: S. audit-key: enhancement|js/game.js|resetGame
- Langage dominant
- JavaScript
- Étoiles
- 4
- Forks
- 2
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de TMHSDigital/Static_Chess
-
documentation good first issue severity: low
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
TMHSDigital/Static_Chess#49 ·
-
documentation good first issue severity: low
Difficulté 2/5 1-3 heures Accessibilité débutants 83/100
TMHSDigital/Static_Chess#48 ·
-
documentation severity: medium
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
TMHSDigital/Static_Chess#47 ·
-
documentation good first issue severity: low
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
TMHSDigital/Static_Chess#46 ·
-
documentation good first issue severity: low
Difficulté 1/5 1-3 heures Accessibilité débutants 82/100
TMHSDigital/Static_Chess#45 ·
Toutes les issues de TMHSDigital/Static_Chess
Issues similaires
-
good first issue tests
Difficulté 1/5 1-3 heures Accessibilité débutants 68/100
Les mainteneurs répondent en général sous 1 jour
-
Download a leaderboard as CSVOuverteenhancement good first issue
Difficulté 2/5 1-3 heures Accessibilité débutants 90/100
-
1️⃣ phase/under-investigation flaky/confirmed flaky/needs-decision type/bug
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
growilabs/growi#12246 · 3 commentaires ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 72/100
TMHSDigital/Free-Game-Dev-Assets#301 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 79/100