midudev / la-velada-web-oficial

Web oficial de La Velada IV de Ibai Llanos
https://lavelada.es
Other
1.39k stars 609 forks source link

fix(#829): Boxer with 4 social link not centered #832

Closed AlejandroSuero closed 7 months ago

AlejandroSuero commented 7 months ago

Descripción

Se hace que solo cree 4 columnas de grid si tiene 4 links

Problema solucionado

Issue #829

Cambios propuestos

  1. Sacar los links a un componente nuevo
  2. Según tengan 5 o menos usar una clase u otra.

Capturas de pantalla (si corresponde)

Antes:

Screenshot 2024-04-02 at 01 41 55

Después:

Screenshot 2024-04-02 at 01 41 45

Comprobación de cambios

Impacto potencial

Mejora del centrado de los links de los boxeadores

Contexto adicional

Enlaces útiles

vercel[bot] commented 7 months ago

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

A member of the Team first needs to authorize it.

AlejandroSuero commented 7 months ago

Si alguien tiene otra propuesta para hacer de cómo centrarlos, es bienvenida.

Había pensado en usar flexbox, pero para seguir el diseño original iba a ser más código y más complicado el dividirlo en columnas, y mirar si es par o impar, para centrar el último link si es impar o quitar una columna.

Edier-Gama commented 7 months ago

Se me ocurre con flex así:

option1

Y ponerle un máximo de w-40 al BoxerSocialLink

option

Queda así, no se que opines

option1_view

option1

github-actions[bot] commented 7 months ago

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

github-actions[bot] commented 7 months ago

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