[ADD] Chip component #98
Conversation
✅ Deploy Preview for egdev6-design-system ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
@andresito87 varias cosas: VISUALIZACIÓN 1- La variante shadow me queda rara con esa sombra en light mode 2-Asegúrate que los elementos cojan los colores correctos en light mode 3-Me queda raro que el icono de close sea tan pequeño (sin hover queda como muy separado) quizás sobreescribir los estilos para que no añada ese padding y el círculo en el hover) 4-El list lo eliminaría para hacerlo molécula en el futuro (aunque es un buen aporte), ya que lo veo más propio de una combinación de varias cosas (noi sólo el chip) 5-Le quitaría los efectos de hover a todas las variantes no interactuables con click ya que el componente en sí queda raro que tenga ese fecto hover cuando sólo es un componente visual e informativo. 6-Revisa errores de accesibilidad: Lo demás lo veo guay. Buen trabajo! |
…se icon in chip component
|
@egdev6 He corregido lo fallos que has propuesto. El único que no me termina de convencer es cuando usamos el componente icon que no se como reescribir sus estilos para que use color claro u oscuro y que no cree hover ni cambios de colores. El problema del punto 2: |
|
1- Revisa los paddings, me quedan muy grandes cuando el chip es un componente que debe ocupar poco espacio. Revisa HeroUI u otras librerías para ajustar 2-Me sigue quedando rara la variante shadow. Prueba a hacerla menos extensa...mira componente button x ej 3- El end content sigue añadiendo fondo de un color diferente 4- los iconos de close me siguen quedando muy pequeños...quizás cambiando los padding (punto1) pueda quedar mejor. Si no ajusta tamaño 5-tienes problemas de accesibilidad con las variantes. Si es el rojo prueba a usar el accent en dark mode o revisa contrastes Lo demás lo veo guay! dale caña! |
@egdev6
PR: Chip - Agregando componente
Nota: Task del backlog movida a done siguiendo las instrucciones Video explicación