-
Notifications
You must be signed in to change notification settings - Fork 2
[ADD] Snippet component #66
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
Conversation
✅ Deploy Preview for egdev6-design-system ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
@egdev6 te mando un draft de mi PR ya que me gustaria saber si voy en la direccion correcta
Estuve documentandome y encontre estos dos patrones que veo que usan heroUI y me parecio interesante asi que lo replique aqui, me gustaria saber si esta bien o necesitarias que lo adapte a como tienen los atomos.
Espero tus comentarios y feedback. referencias |
|
@Megajjks te comento: Visualmente:
la forma de usarlos con tailwidn es 'bg-orange-light' para backgrounds 'border-orange-lighti para bordes y 'text-orange-light' para textos (los tienes en src/components/utils/types/safelist.ts)
Pull request (código):
|
|
@Megajjks cuando lo tengas listo mencióname por aquí y le pegamos otra vuelta ;) |
|
Buenas, me gustaría comentar algo de los commits de la PR. El commit de merge main en tu rama (c4f8727) no debería estar. Si haces rebase, evitas que se hagan commits de merge y la rama queda limpia, evitando posibles conflictos en otras ramas que puedan hacer tambien merge en vez de rebase. El commit de fix del linter (46a02f9) no deberías incluirlo en una rama de feature. Todos los demás commits de tu rama realmente deberían ser uno solo si no hay una razón específica para hacer más de uno, así facilita la revisión de la PR y la limpieza de la rama main. Esto es solo mi opinión, ¿vosotros qué pensáis? |
|
@Megajjks Me parece correcto lo que comenta @AmosVelez. Bien visto! Revisalo y si tienes dudas comentamos y resuelve los conflictos con los últimos combio de main |
|
@AmosVelez gracias por la observacion, se me paso el preformateo del linter. en los siguientes cambios que me baje del main lo hare con rebase para no generar problemas @egdev6 |
2060f9d to
5f6c6eb
Compare
|
@egdev6 cambié el PR de Draft a ready for approve. Considero una buena practicar para cuando el ticket que se está desarrollando lleva mucho tiempo generar PR draft el cual permite que revises el código y cuando ya se tenga listo el PR cambiarlo a Ready for Approve, ayuda mucho para ver los avances de los demás y estos puedan hacer code review con el fin de mejorar el tiempo y no hacer cansado el code review. |
|
@Megajjks. Te paso feedback: Visualmente: 1-Ojo con el modo light. Habría que cambiar colores para que el contraste sea correcto 2- La variante large em queda muy grande...quizás podrías aumentar tamaño de fuente pero mantener los paddings 3- revisa que pese a cambiar de tamaño siga alineado con el contenido 4-Para el uso de los grises puedes tomar de referencia el componente Input (tanto light como dark) para que vayan alineados con otros componentes Código nada que objetar. Lo veo guay! Buen curro! |
|
Va que va, gracias por el feedback, voy trabajando en ello, gracias @egdev6 |
…nd improve aligment styles
|
@egdev6 ya esta listo los cambios solicitados
|
|
Super top @Megajjks! currazo! mergeamos a main y vamos a por el siguiente! |






✨ Feature: Add Snippet molecule component
Description
This PR introduces the new
Snippetmolecule component, including its base structure, props, and logic. The component supports slot-based styling, copy-to-clipboard functionality, and is fully documented with Storybook stories.Additionally, the slot styling scheme was refactored to use class-variance-authority (CVA), enabling scalable and maintainable variants for size, color, and style.
Changes
Snippet.tsxfor the main component logic and rendering.SnippetPropsand slot variants intypes.tsusing CVA for flexible configuration.useSnippet.tshook for handling copy logic and slot classes, now using CVA for slot styling.Snippet.tsxto use CVA-based classes.Snippet.stories.tsx) to showcase usage and variants.index.ts.[var(--color-primary)]for better theme support.Motivation
The new Snippet component provides a reusable, accessible, and customizable way to display code snippets with copy functionality, aligning with the design system's goals.
Refactoring to CVA makes it easier to extend and maintain styling variants for future needs.
The color fix ensures better integration with custom themes.
Checklist
Additional Notes
Feedback and suggestions for further improvements are