AlejandroSuero commented 7 months ago

Descripción

El contador estaba a 090 en vez de a 99.

Problema solucionado

El contador ahora muestra los días de forma dinámica.

Closes #858

Cambios propuestos

Antes:

https://github.com/midudev/la-velada-web-oficial/blob/88e67821ee0bbe3b4727fb9671da0c611cca2ad5/src/sections/Countdown.astro#L88-L96

Lo días se mostraban days = [9,9] por tanto fallaba la animación

Ahora:

if (
  $days.firstDigit instanceof HTMLElement &&
  $days.secondDigit instanceof HTMLElement &&
  $days.thirdDigit instanceof HTMLElement
) {
  switch (days.length) {
    case 3:
      animateDigit($days.firstDigit, days[0], "day")
      animateDigit($days.secondDigit, days[1], "day")
      animateDigit($days.thirdDigit, days[2], "day")
    break
    case 2:                          
      $days.firstDigit.parentNode?.removeChild($days.firstDigit)
      animateDigit($days.secondDigit, days[0], "day")
      animateDigit($days.thirdDigit, days[1], "day")
    break
    default:                             
      $days.secondDigit.parentNode?.removeChild($days.secondDigit)
       animateDigit($days.thirdDigit, days[0], "day")
  }
}

Se añade un switch para comprobar la longitud de days y se elimina el que sobra

Capturas de pantalla (si corresponde)

Antes:

Ahora:

https://github.com/midudev/la-velada-web-oficial/assets/71392160/13fcd4a1-858b-41ee-828e-8bbe66bce9fc

Comprobación de cambios

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

Impacto potencial

Mejorar la experiencia de usuario y solucionar bugs visuales.

Contexto adicional

858

Enlaces útiles

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

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.