-
Notifications
You must be signed in to change notification settings - Fork 5
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
Use panda for icons #2200
base: master
Are you sure you want to change the base?
Use panda for icons #2200
Conversation
import { defineTokens } from "@pandacss/dev"; | ||
|
||
export default defineTokens.spacing({ | ||
1: { value: "2px" }, |
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.
Her dobler du px fra tokenet. Hadde det vore enklere å ha samme token som pixelverdi?
to ensure that global css never "wins" w.r.t specificity
4aab733
to
a395f44
Compare
a395f44
to
713670f
Compare
import { defineConfig } from "@pandacss/dev"; | ||
import { preset } from "@ndla/primitives"; | ||
|
||
export default defineConfig({ |
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.
Må vi ha en sånn her config i hver pakke? burde vi hatt en base-config som kan gjenbrukes?
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.
Det kan vi sikkert!
936446b
to
f7d946a
Compare
3fd2abe
to
6aa5900
Compare
9ebb053
to
7ee6b80
Compare
aaf8bfb
to
4ca5f63
Compare
Egentlig bare et eksempel på hvordan primitives kan brukes. Realistisk sett kommer vi ikke til å reeksportere IconProps, og heller ikke Spinner. Ville bare ikke innføre noe som var kjempebreaking.