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

feat(Gallery, UX): added swipe animation for mobile users #960

Closed

Conversation

AlejandroSuero
Copy link
Contributor

Descripción

Se añade soporte para dispositivos táctiles en Gallery.astro para pasar de foto.

Problema solucionado

En dispositivos móviles, tablet o similares, al no aparecer las flechas para pasar de foto, había que cerrar y volver a elegir foto. Con esta PR se añade una manera intuitiva de pasar de foto usando gestos de movimiento touch{start|move|end}.

Cambios propuestos

  1. Añadir clase swipe-element para la imagen que va a ser animada.
  2. Añadir eventListener("touchstart", swipeImage, { passive: true }) con la opción de passive: true para mejorar rendimiento de estos eventos.
  3. Añadir función swipeImage en la cuál se realizará la animación y la lógica para saber si hay que mostrar la siguiente image o la anterior a la actualmente mostrada.
  4. Añadir eventListener("touchmove", ...) y evenListener("thouchend", ...) dentro de la función swipeImage.

Capturas de pantalla (si corresponde)

Screen.Recording.2024-05-28.at.13.51.41.mov

Note

También he probado los cambios en mi dispositivo móvil haciendo uso de pnpm dev --host.

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

Mejoras de experiencia para usuarios de dispositivos táctiles como móvil, tablet, etc.

Contexto adicional

He probado a no realizar ninguna animación indicativa del swipe pero creo que el usuario llega a entender mejor que movimiento se está realizando si se muestra la imagen deslizándose y desapareciendo conforme se realiza la acción.

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 closed this Nov 4, 2024
@AlejandroSuero AlejandroSuero deleted the feature/add-swipe-animation branch November 5, 2024 18:20
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