-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
feat(FluidTextArea): adds new fluid text area #12053
feat(FluidTextArea): adds new fluid text area #12053
Conversation
✅ Deploy Preview for carbon-components-react ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site settings. |
✅ Deploy Preview for carbon-elements ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
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.
Looking good just a few things
Component
- Should the resize handle feature also be disabled in the disabled state?
- Error + focus state: when an input with an error state receives focus the border should turn blue/
$focus
to indicate its been given focus. This is what we do across invalid states with the default inputs.
- As of Friday, I added in a Skeleton state for all the fluid input. It was an oversite in the original spec, so can we add that in now?
Storybook
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.
One small change, otherwise looks great! That playgroundWidth
control is a super cool idea 🔥
packages/react/src/components/FluidTextArea/FluidTextArea.stories.js
Outdated
Show resolved
Hide resolved
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 didn't see the storybook change for the background color in the layers example but the component itself looks good to go.
@aagonzales yea i think theres an issue to look at with layers and adding backgrounds unrelated to text area. so we will take a look at that in a separate issue |
Closes #12020
Closes #12018
Adds fluid text area