-
Notifications
You must be signed in to change notification settings - Fork 3
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
✨(react) add TextArea component #199
Conversation
🦋 Changeset detectedLatest commit: 56c8138 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
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 |
c85a983
to
fd6d395
Compare
"border-radius": "8px", | ||
"border-radius--hover": "2px", | ||
"border-radius--focus": "2px", | ||
"border-width": "1px", |
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.
I realized we are defining those tokens for each inputs that seems weird. This is not in the scope of this PR, but I think there is something to improve here.
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.
Yes I agree, we have to keep in mind that we still want to guarantee the possibilty to customize them for each individual inputs, but in the mean time those are always the same .. 🤔
fd6d395
to
fbb3958
Compare
14c895f
to
aca28b4
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.
LGTM ✅
How could we ship a design system library without a textarea?
This is made to showcase the TextArea in real world use cases.
This typo was probably due to a bad copy pasta.
The font family of the design system was not applied to input value.
aca28b4
to
56c8138
Compare
How could we ship a design system library without a textarea?