feat(ai-chat-log): Adding docs for new component #3963
Conversation
…paste into docs/ai-logs
| <ThumbsDownIcon decorative={false} title="dislike result" /> | ||
| </Button> | ||
| </AIChatMessageActionCard> | ||
| <AIChatMessageActionCard> |
There was a problem hiding this comment.
suggestion: I think I added some aria labels to the rewrite button and maybe also the thumbs up/down buttons in the stories - can you copy whatever I did there for the action card sections in these examples?
There was a problem hiding this comment.
Great catch. I will address this in the next commit
| }} | ||
| language="jsx" | ||
| > | ||
| {`<AIChatLog> |
There was a problem hiding this comment.
question (non-blocking): just curious, nothing wrong with it, but why are you writing this example out here and importing the others?
There was a problem hiding this comment.
The first LivePreview in the chat-log index page did the same. Seeing a trend for simpler components while looking through other doc pages not being put in examples.
I open to changing this if we want to start a best practice on it
There was a problem hiding this comment.
Makes sense, no need to change :)
| @@ -292,7 +292,7 @@ This example combines all the separate features displayed previously into one ex | |||
|
|
|||
| The `useChatLogger` hook provides a hook based approach to managing chat state. It is best used with the `<ChatLogger />` component. | |||
|
|
|||
| `useChatLogger` returns 3 things: | |||
| `useChatLogger` returns 4 things: | |||
| language="jsx" | ||
| noInline | ||
| > | ||
| {botWithBodyActions} |
There was a problem hiding this comment.
issue(blocking): these buttons need some padding, can you also go down to just 3 buttons so we're not showing off the wrapping bug until we fix it 😸
packages/paste-website/src/pages/components/ai-chat-log/index.mdx
Outdated
Show resolved
Hide resolved
packages/paste-website/src/pages/components/ai-chat-log/index.mdx
Outdated
Show resolved
Hide resolved
packages/paste-website/src/pages/components/ai-chat-log/index.mdx
Outdated
Show resolved
Hide resolved
Co-authored-by: Sarah <sali@twilio.com>
krisantrobus
added
🚀 merge it!
🕵🏻♀️ Run website visual regression
Passing run #8153 ↗︎
Details:
Review all test suite changes for PR #3963 ↗︎ |
|||||||||||||||
Added new docs page for
ai-chat-log