Skip to content

Corregida UI móvil - #51

Merged
eeminionn merged 5 commits into
eeminionn:mainfrom
ChileCris2011:main
Aug 12, 2026
Merged

Corregida UI móvil#51
eeminionn merged 5 commits into
eeminionn:mainfrom
ChileCris2011:main

Conversation

@ChileCris2011

Copy link
Copy Markdown
Contributor

Issue relacionado

Closes #50

Qué cambia

La UI móvil, en la página de resolución de misiones, el contenedor de la pestaña "Enunciado" ahora responde correctamente a los gestos de teléfono

Por qué

Las cosas del apartado "Enunciado" no respondían a ningún gesto en móvil (pantallas con menos de 960px de ancho), ya sea deslizar para seguir leyendo, cambiar pestañas, e incluso movimientos con ratón.

Implementación

La clase .workbench-panes se colocaba encima de la clase .brief-pane (El 'contenedor' de la explicación), haciendo que 'robara' todos los gestos dirigidos a la clase correspondiente. Se modifica v2/styles.css para ocultar .workbench-panes cuando se usa la UI móvil, así como v2/e2e/mobile.spec.ts para cubrir botones y scrolls

Evidencia visual

Validación

  • pnpm check
  • pnpm test:e2e
  • supabase test db o no aplica (no aplica)
  • Probé el recorrido afectado en local o beta

Privacidad, seguridad y accesibilidad

  • No añadí secretos, soluciones privadas ni datos personales al código, fixtures o logs.
  • Revisé RLS/permisos si cambié acceso a datos o autenticación.
  • Revisé teclado, nombres accesibles, contraste y responsive si cambié la interfaz.
  • Documenté cualquier riesgo residual.

Despliegue

Se puede implementar al instante, sin necesidad de cambiar nada.

Checklist de entrega

  • El PR tiene responsable, labels y milestone cuando corresponde. (Eso no existe en PRs...)
  • Actualicé pruebas y documentación. (huh?)
  • No contiene cambios ajenos al issue.
  • CI está en verde. (Falló Supabase, al ser solo frontend 😒)
  • Está listo para revisión.

ChileCris2011 and others added 5 commits August 10, 2026 21:24
Reposiciona '.workbench-panes' en la página de misiones para que '.brief-pane' vuelva a obtener los gestos
Añade nuevo botón "Test manual"
@eeminionn
eeminionn merged commit ad3e067 into eeminionn:main Aug 12, 2026
3 checks passed
@eeminionn

Copy link
Copy Markdown
Owner

nice, @ChileCris2011 quizas cambiare el compilador a alguna API de algun compilador online, asi que si lo hago te avisare por si puedes checkear que no se rompa de nuevo la UI de celular

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Pestaña "Enunciado" de misiones no responde en teléfono

2 participants