[WIP] Add Calendar component closes #831, #483

AlejandroSuero commented 5 months ago

Descripción

He notado que en el algunos dispositivos móviles, se ralentiza un poco a la hora de hacer la carga (en el mío por ejemplo, y emulando algunos dispositivos de hace 2-3 años atrás).

También si se quiere modificar, como ya se ha hecho previamente, al estar add-to-calendar.{js,css} minificado, se crean muchos conflictos y resulta difícil a la hora de modificar, debido a los nombres de variables que tiene.

Problema solucionado

Issues:

831
483

Cambios propuestos

Quitar el minificado tanto de js como de css
Crear un componente más personalizable
Ajustar la carga de datos a lo imprescindible

Capturas de pantalla (si corresponde)

[!important] El color del botón es para hacer una distinción

Antes:

https://github.com/midudev/la-velada-web-oficial/assets/71392160/59f48536-9f46-4321-b8af-a3322b3fe653

Después:

https://github.com/midudev/la-velada-web-oficial/assets/71392160/f08e19d6-aee9-4cfb-b6d7-e9a48b3b2862

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

Reducir ralentización en dispositivos de menos recursos como algunos móviles u ordenadores no tan potentes.

Contexto adicional

Enlaces útiles

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

vercel[bot] commented 5 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 5 months ago

A falta de implementar:

[x] Detectar si está usando móvil u ordenador
- [X] Añadir .ics file cuando sea móvil
[ ] Usar etiqueta <dialog>
[ ] ¿Iconos para los distintos calendarios?

AlejandroSuero commented 5 months ago

@midudev @ElStron @Daniel-Lagos

Si no es mucha molestia, clonad mi repo y usad la rama feature/add-Calendar-component para comprobar que os funcione tanto en móvil como en navegador?

Yo he comprobado que funcionan desde MacOSX, Linux y Windows, pero no he podido comprobarlo en móvil.

Sé que por ejemplo en este proyecto que hice 3D-product-customization, puedo hacer que se descargue la imagen generada en móvil desde el deploy que tengo https://aome3dcustomizer.vercel.app/. He utilizado la misma lógica para descargar el archivo .ics, pero no sé si se hace correctamente.

midudev / la-velada-web-oficial