-
Notifications
You must be signed in to change notification settings - Fork 4
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
common: Add new component UI library #284
base: main
Are you sure you want to change the base?
Conversation
🦋 Changeset detectedLatest commit: 2d9c9b2 The changes in this PR will be included in the next version bump. This PR includes changesets to release 2 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
c4590e9
to
1fcfa6b
Compare
4c62ad5
to
597827f
Compare
597827f
to
2d9c9b2
Compare
@@ -92,4 +125,19 @@ const spacing = { | |||
10: "192px", | |||
}; | |||
|
|||
export { colors, fontWeights, fontSizes, lineHeights, borderRadius, dropShadow, spacing }; | |||
const useTheme = () => { | |||
const theme = useEmotionTheme(); |
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.
not sure how themes with the css
prop will work in cases where there's no emotion provider
Maybe we can just avoid using it
<Thing css={(theme) => ???}/>
Description
Release new
react-ui
component library, and use it inbrowser-sdk
. It's pretty minimal at the moment, with only a couple of icons, andAvatar
andPopover
components.Summary:
Related Issue:
Testing
yarn build && yarn dev
Screenshots/GIFs (if applicable)
Checklist
under the terms of the MIT license.
Additional Information