-
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(ChatButton): add in ChatButton
component
#15675
Conversation
✅ Deploy Preview for v11-carbon-react ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Visually, this looks good! I just want to confirm we've got the interaction of the new state correct. Now that I see it, I'm wondering if it should actually be more of a read-only instead of disabled. |
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.
If the chat devs are good with this, then I think its good to go 👍
6535b04
to
cfb1674
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!
Co-authored-by: Alison Joseph <[email protected]>
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! 🚀
0eac7a8
Hey there! v11.50.0 was just released that references this issue/PR. |
Closes #15621
Adds in the
ChatButton
component, which is just a wrapper aroundButton
that restricts the component adhere to AI stylesChangelog
New
unstable__ChatButton
andunstable__ChatButtonSkeleton
componentsChatButton
Testing / Reviewing
Ensure the new component matches the spec