You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
En esta solicitud de extracción, se ha añadido un gestor de iconos y se han reemplazado los iconos en Count Down, el footer, y la main date con el gestor de iconos.
Problema solucionado
Anteriormente, los iconos en Count Down, el footer, y la main date estaban siendo manejados individualmente. Esta solicitud de extracción aborda este problema al introducir un gestor de iconos que permite un manejo más eficiente y centralizado de los iconos.
Cambios propuestos
Los cambios específicos realizados en el código son los siguientes:
Añadir un gestor de iconos.
Reemplazar los iconos en Count Down, el footer, y la main date con el gestor de iconos.
Estos cambios permiten un manejo más eficiente y centralizado de los iconos en la aplicación.
Comprobación de cambios
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.
Impacto potencial
Estos cambios deberían mejorar la eficiencia y la mantenibilidad del código al centralizar el manejo de los iconos. No se anticipan problemas de compatibilidad o cambios en el rendimiento como resultado de estos cambios.
Antes de nada muchas gracias por contribuir! Y pedazo de componente, ayuda mucho a la hora de leer la semántica y así tenemos centralizados todos los iconos. No te la apruebo aún porque no puedo revisarla pero mañana por la mañana le hecho un vistazo y si todo esta en orden le hago el merge.
Muchas gracias por contribuir y sigue moviendo las manitas!! 🚀
Lo único que me gastaría saber es si al estar importando los iconos arriba, en el caso de en un futuro de 20 iconos. Si luego en una página solo usamos 1, supongo que estaríamos haciendo que el componente pesase más y cargase esos 20 iconos innecesariamente. (lo miro a cara futuro claro).
A lo mejor una alternativa seria que simplemente los iconos que tuviesemos a mano como directamente en el proyecto, los pasasemos a componente (como ya estaba hecho con alguno si no me equivoco).
Igualmente etiqueto por aquí a @midudev para que confirme si lo que digo es correcto ya que no tengo mucha experiencia con Astro y no se si lo que digo es 100% así.
El acabará de valorar esta PR. Muchísimas gracias por contribuir!! 🚀
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Descripción
En esta solicitud de extracción, se ha añadido un gestor de iconos y se han reemplazado los iconos en
Count Down, elfooter, y lamain datecon el gestor de iconos.Problema solucionado
Anteriormente, los iconos en
Count Down, elfooter, y lamain dateestaban siendo manejados individualmente. Esta solicitud de extracción aborda este problema al introducir un gestor de iconos que permite un manejo más eficiente y centralizado de los iconos.Cambios propuestos
Los cambios específicos realizados en el código son los siguientes:
Count Down, elfooter, y lamain datecon el gestor de iconos.Estos cambios permiten un manejo más eficiente y centralizado de los iconos en la aplicación.
Comprobación de cambios
Impacto potencial
Estos cambios deberían mejorar la eficiencia y la mantenibilidad del código al centralizar el manejo de los iconos. No se anticipan problemas de compatibilidad o cambios en el rendimiento como resultado de estos cambios.