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: ForecastBoxer texts overflow #959

Merged
merged 1 commit into from
May 27, 2024

Conversation

dresandev
Copy link
Contributor

Descripción

Se corrige contenido cortado en el componente ForecastBoxer

Cambios propuestos

Se remueve el overflow-hidden del wrapper para evitar que se corte el contenido que sobresale o que puede llegar a sobresalir, en este caso, el nombre del boxeador y el porcentaje. El overflow-hidden se usaba para mantener el aspect-ratio de la imagen del boxeador, ya que la imagen crecía con el contenido. Para solucionar esto, se usa absolute en el contenedor del título y porcentaje, posicionándolo debajo y al centro de la imagen, y se agrega al wrapper un pb-[content-height]. De esta manera, al "no tener" el contenido en el wrapper, la imagen mantiene su aspect ratio y el contenido no se corta.

Capturas de pantalla

Antes:

image

Después:

image

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.

Copy link

vercel bot commented May 22, 2024

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

A member of the Team first needs to authorize it.

@midudev midudev merged commit 873a47f into midudev:main May 27, 2024
0 of 2 checks passed
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.

2 participants