Corregida UI móvil - #51
Merged
Merged
Conversation
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"
This reverts commit 25f222c.
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 |
7 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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-panesse 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 modificav2/styles.csspara ocultar.workbench-panescuando se usa la UI móvil, así comov2/e2e/mobile.spec.tspara cubrir botones y scrollsEvidencia visual
Validación
pnpm checkpnpm test:e2esupabase test dbo no aplica (no aplica)Privacidad, seguridad y accesibilidad
Despliegue
Se puede implementar al instante, sin necesidad de cambiar nada.
Checklist de entrega