Pause settings page shows a stale SOUND label after pressing Mute

Aberta Para iniciantes
  • Algo errado?
#105 0 comentários 0 reações 0 responsáveis Ver no GitHub

Ninguém assumiu esta issue ainda.

Avaliação

Dificuldade
1/5
Tempo estimado
Menos de uma hora
Facilidade para iniciantes
88/100
Tipo de issue
Bug
Clareza
Claramente especificada
Status de atividade
Ativa
Stack de tecnologia
typescript
Domínio
frontend, game-dev

Direção de pesquisa

Comece em shared/pause.ts no ramo de mute, por volta das linhas 142-145, que chama apenas updateMuteText(), depois leia refreshSettings(), por volta das linhas 233-246, para ver quais dos cinco rótulos ele redesenha. A mudança provável é chamar também this.refreshSettings() nesse ramo, e shared/settings.ts mostra de onde vem o estado de mudo. Está pronto quando o rótulo SOUND na página de configurações muda assim que Mute é pressionado; verifique com npm test e abrindo o menu de pausa em um jogo em execução.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

bug good first issue severity: low

Summary

With the pause menu's SETTINGS page open, pressing the Mute key or button changes the setting but the menu keeps showing the old SOUND ON / SOUND OFF label.

Where

  • shared/pause.ts:142-145 (the mute branch only calls updateMuteText())
  • shared/pause.ts:233-246 (refreshSettings() is only called from change() and showPage('settings'))

Details

if (all.some((c) => c.justPressed('mute'))) { setMuted(!isMuted()); this.updateMuteText(); }

The corner "MUTED (M)" text updates, but the settings labels are not redrawn.

Steps to reproduce

  1. Pause any game and choose SETTINGS.
  2. Press M. "MUTED (M)" appears in the corner while the menu still says SOUND ON.

Suggested fix

Call this.refreshSettings() in the mute branch as well (it only sets five labels). Alternatively, subscribe the controller to onSettingsChange and refresh whichever page is visible, which also covers changes made in another tab.

Acceptance criteria

  • The SOUND label matches the mute state immediately after pressing Mute on the settings page.

Getting started

  • Relevant files: shared/pause.ts, shared/settings.ts
  • How to run tests: npm test

Filed by repo-audit/full-audit at de2b25b. Severity: low. Effort: S. audit-key: bug|shared/pause.ts|mute-refresh

Linguagem predominante
TypeScript
Estrelas
3
Forks
3
Merge médio
8h 13min
PRs com merge (30d)
2

Preparar o ambiente

Este projeto não oferece contêiner de desenvolvimento, Dockerfile nem guia de contribuição, então a configuração fica por sua conta: comece pelo README e veja nosso guia da primeira contribuição para os passos gerais.

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de TMHSDigital/GH-Arcade

Todas as issues de TMHSDigital/GH-Arcade

Issues semelhantes

Mais issues de TypeScript

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.