Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(#911): fixed /combates -> /combates/[id] image size #915

Merged

Conversation

AlejandroSuero
Copy link
Contributor

@AlejandroSuero AlejandroSuero commented Apr 18, 2024

Descripción

Como se comenta en #911, en algunos dispositivos se veía mal u ocupaba mucho espacio.

Problema solucionado

Se mantiene una coherencia entre dispositivos.

Cambios propuestos

  1. Que las imágenes ocupen el mismo alto en todos los dispositivos.
  2. En móvil y tablet, reducir el tamaño para que pueda verse el vídeo y el título con mayor claridad.
  3. Eliminar el renderizado de las imágenes si es REY_DE_LA_PISTA_ID al no existir las imágenes para ello.
Screenshot 2024-04-18 at 18 18 37

Capturas de pantalla (si corresponde)

Screenshot 2024-04-18 at 14 49 33 Screenshot 2024-04-18 at 14 50 07 Screenshot 2024-04-18 at 14 50 16 Screenshot 2024-04-18 at 14 50 29

Comprobación de cambios

  • He revisado que no haya ninguna PR (pull request) ya abierta con un problema similar, siguiendo el apartado de buenas prácticas
  • He revisado localmente los cambios para asegurarme de que no haya errores ni problemas.
  • He probado estos cambios en múltiples dispositivos y navegadores para asegurarme de que la landing page se vea y funcione correctamente.
  • He actualizado la documentación, si corresponde.

Impacto potencial

Mejorar la interfaz para mayor adaptabilidad a dispositivos.

Contexto adicional

Closes #911.

Enlaces útiles

  • Documentación del proyecto:
  • Código de referencia:

Copy link

vercel bot commented Apr 18, 2024

@AlejandroSuero is attempting to deploy a commit to the midudev pro Team on Vercel.

A member of the Team first needs to authorize it.

Copy link

⚠️ Esta Pull Request tiene conflictos. Por favor, resuelvelos antes de que podamos evaluar los cambios.

Copy link

⚠️ Esta Pull Request tiene conflictos. Por favor, resuelvelos antes de que podamos evaluar los cambios.

Copy link

⚠️ Esta Pull Request tiene conflictos. Por favor, resuelvelos antes de que podamos evaluar los cambios.

Copy link

⚠️ Esta Pull Request tiene conflictos. Por favor, resuelvelos antes de que podamos evaluar los cambios.

Copy link

⚠️ Esta Pull Request tiene conflictos. Por favor, resuelvelos antes de que podamos evaluar los cambios.

@midudev midudev merged commit ce98a32 into midudev:main May 14, 2024
1 of 3 checks passed
@AlejandroSuero AlejandroSuero deleted the feature/fix-combat-image-transition branch May 14, 2024 00:31
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.

No se visualiza bien el detalle de los combates en monitores más pequeños como notebooks
2 participants