Confirm before New Game discards a game in progress
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 2/5
- Tempo estimado
- 1-3 horas
- Facilidade para iniciantes
- 78/100
- Tipo de issue
- Funcionalidade
- Clareza
- Claramente especificada
- Status de atividade
- Ativa
- Stack de tecnologia
- javascript
- Domínio
- frontend
Direção de pesquisa
Leia js/game.js:41-42 e js/game.js:76-91 para rastrear como New Game chama resetGame e, em seguida, examine index.html:58-61 para ver os controles adjacentes. Escolha uma das duas abordagens propostas e confira como o estado do jogo e o histórico de jogadas são armazenados. O trabalho estará concluído quando uma partida em andamento não puder ser descartada com um único toque, enquanto iniciar uma nova partida sem jogadas ou após o fim do jogo não exigir nenhuma etapa extra.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
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
- Linguagem predominante
- JavaScript
- Estrelas
- 4
- Forks
- 2
- Métricas de merge de PRs
- Nenhum PR com merge em 30d
Preparar o ambiente
- Sem Dockerfile nem arquivo Docker Compose
- Tem um modelo de pull request
- Ler o guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de TMHSDigital/Static_Chess
-
documentation good first issue severity: low
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 72/100
TMHSDigital/Static_Chess#49 ·
-
documentation good first issue severity: low
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 83/100
TMHSDigital/Static_Chess#48 ·
-
documentation severity: medium
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
TMHSDigital/Static_Chess#47 ·
-
documentation good first issue severity: low
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 82/100
TMHSDigital/Static_Chess#46 ·
-
documentation good first issue severity: low
Dificuldade 1/5 1-3 horas Facilidade para iniciantes 82/100
TMHSDigital/Static_Chess#45 ·
Todas as issues de TMHSDigital/Static_Chess
Issues semelhantes
-
good first issue tests
Dificuldade 1/5 1-3 horas Facilidade para iniciantes 68/100
Mantenedores costumam responder em até 1 dia
-
enhancement good first issue
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 90/100
-
1️⃣ phase/under-investigation flaky/confirmed flaky/needs-decision type/bug
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 65/100
growilabs/growi#12246 · 3 comentários ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 72/100
TMHSDigital/Free-Game-Dev-Assets#301 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 79/100