-
Notifications
You must be signed in to change notification settings - Fork 9
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
Inline the avatar mark to remove the react-dom/server dependency #193
Conversation
Deploying compound-web with Cloudflare Pages
|
fd5fd47
to
643929a
Compare
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.
This is a step towards making it harder to simply diff files between Figma and git, or better yet automate them for design to own them
643929a
to
828459a
Compare
I would agree for icons in the design tokens, but it feels like this path was manually crafted? |
React-SDK had a similar mask (before switching to Compound Avatars) which came from Figma |
Fine, I replaced that with a raw import of the SVG. I guess the bundle size would be slightly bigger as the SVG has comments in it, but it's nothing compared to react-dom/server |
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.
Looks sane
This unnecessarily adds a react renderer to the bundle/dependencies just for a simple SVG that doesn't change.
It's also a step towards removing SVGR from this project.