-
Notifications
You must be signed in to change notification settings - Fork 0
Input #26
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
Input #26
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ciao! Grazie mille per la PR, molto precisa, come previsto ti faccio un po' di nitpicking per fare tutto preciso.
Una cosa che abbiamo notato per quanto riguarda il rounded-buttonM che non funziona in tailwind: avendo il glass già la classe rounded-lg, tailwind-merge non sapeva come unire le cose dato che la nostra è custom, stando alla loro documentazione va estesa la funzione cn in `@/lib/utils.ts/ così:
import { type ClassValue, clsx } from "clsx"
import { extendTailwindMerge } from "tailwind-merge"
const twMerge = extendTailwindMerge({
extend: {
theme: {
radius: ["images", "rectangles", "buttonsM", "buttonsL"], // custom rounded classes from figma
},
},
})
export function cn(...inputs: ClassValue[]) {
return twMerge(clsx(inputs))
}Prova a modificare la cosa in questo modo e guarda se buttonM funziona come previsto.
Potrebbe essere che sarà necessario in futuro fare cose simili anche per i colori di Figma, ma non ho verificato. Apro io una issue per questa cosa.
|
Nice ottimo lavoro, bravissima LGTM 👍 |
Closes #13