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(Gallery): boxer images flickering when updating gallery #961

Merged

Conversation

AlejandroSuero
Copy link
Contributor

Descripción

Como se menciona en #918, al hacer la actualización de imágenes en Gallery.astro se utiliza View Transitions API lo cual genera que las imágenes que tienen una transición actúen.

Problema solucionado

Fixes #918

if ($lightbox.open && document.startViewTransition) {
document.startViewTransition(() => {
updateImg(link)
})
return
} else {
updateImg(link)
}

Cambios propuestos

  1. Eliminar la transición usando View Transitions API
- if ($lightbox.open && document.startViewTransition) {
-   document.startViewTransition(() => {
-     updateImg(link)
-   })
-  return
- } else {
+ if ($lightbox.open) {
    updateImg(link)
  }

Capturas de pantalla (si corresponde)

Antes:

Gallery-image-flickering.mp4

Note

En #918 se explica más en detalle el parpadeo en cuestión.

Después:

Gallery-fixed-flickering.mp4

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 experiencia de usuario

Contexto adicional

Si se quiere mantener la animación que se creaba con View Transitions API o generar una nueva, se podría añadir una animación CSS y comprobar cuando se empieza y termina con TypeScript dando así soporte a los navegadores sin View Transitions API.

Enlaces útiles

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

Copy link

vercel bot commented May 28, 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.

@midudev midudev merged commit a411c04 into midudev:main May 29, 2024
0 of 2 checks passed
@AlejandroSuero AlejandroSuero deleted the feature/fix-boxer-image-flickering branch May 29, 2024 13:28
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.

E NOTADO QUE EN CHROME HACE UN PARPADEO AL MOMENTO DE PASAR LAS IMAGENES SE VE RARO
2 participants