Share HUD text setup and move Breakout to shared particle helpers

Abierto
#42 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
3/5
Tiempo estimado
1-2 días
Aptitud para principiantes
72/100
Tipo de issue
Refactorización
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
typescript

Línea de trabajo

Read shared/ui.ts:101-122 and the listed game scene locations to compare the repeated HUD setup and existing shared particle helpers. Start with games/breakout/scenes/BootScene.ts:27-28 and games/breakout/scenes/GameScene.ts:115; use npm run typecheck and npm run build to check the changes. Done when HUD text styling is defined once and Breakout uses sparkTexture() and burstEmitter().

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

severity: low tech-debt

Summary

The same HUD text style literal and score text setup are repeated in seven game scenes plus the template, with different depths for the same element. Breakout builds its own spark texture and burst emitter while the other nine games use the shared helpers.

Where

  • For example games/astro/scenes/GameScene.ts:144-145, games/hop/scenes/GameScene.ts:104-105, games/siege/scenes/GameScene.ts:154-155
  • games/breakout/scenes/BootScene.ts:27-28, games/breakout/scenes/GameScene.ts:115
  • shared/ui.ts:101-122

Impact

A HUD restyle (font size, text colour for another palette, safe-area offset) is an eight-file edit, and Breakout is missed by any change to the shared particle helpers.

Suggested fix

Add HUD_STYLE and hudText(scene, x, y, text, size?) to shared/ui.ts and migrate one game per commit. Switch Breakout to sparkTexture() and burstEmitter().

Acceptance criteria

  • HUD text style is defined once
  • Breakout uses the shared particle helpers

Getting started

  • Relevant files: shared/ui.ts, games/breakout/scenes/BootScene.ts, games/breakout/scenes/GameScene.ts
  • Run locally: npm install, then npm run dev
  • Checks: npm run typecheck and npm run build (there is no test suite yet)
  • Related docs: README.md

Filed by repo-audit/full-audit at b001f5d. Severity: low. Effort: S. audit-key: tech-debt|shared/ui.ts|hudText

Lenguaje dominante
TypeScript
Estrellas
3
Forks
3
Métricas de merge de PR
Sin PR fusionados en 30 d

Preparar el entorno

Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de TMHSDigital/GH-Arcade

Todos los issues de TMHSDigital/GH-Arcade

Issues similares

Más issues de TypeScript

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.