feat(Message): add default bot message avatar#850
Conversation
|
Preview: https://chatbot-pr-chatbot-850.surge.sh A11y report: https://chatbot-pr-chatbot-850-a11y.surge.sh |
thatblindgeye
left a comment
thatblindgeye
left a comment
There was a problem hiding this comment.
Codewise this all looks pretty good to me, just a couple comments beloww. Will re-review once we're able to pull in the latest versions.
Once we do pull in latest versions, could we also update the "User messages" example to show how a consumer can pass in initials+color to a user message? Maybe after the second message in that example, "This is a user message with avatarProps set to add a border." I dont think we need to show all the colors, though.
kmcfaul
force-pushed
the
chatbot-bot-icon
branch
from
58b265c to
b69ea93
Compare
mcoker
left a comment
mcoker
left a comment
There was a problem hiding this comment.
---------------
| bleep bloop |
---------------
/
🤖
|
Now we just wait for official release numbers 😎 |
| /** Name of the user */ | ||
| name?: string; | ||
| /** Avatar for the user. Pass a string for an image src, or a ReactNode for custom content like an svg or icon. */ | ||
| /** Avatar for the user. Pass a string for an image src, or a ReactNode for custom content like an svg or icon. Avatars for Messages withrole="bot" are pre-defined and not customizable. */ |
There was a problem hiding this comment.
I think you're missing a space between with and role!
6 participants
Closes: #816
Todos -
avatarsrc prop from all bot message examplesOpen question -
Messageor would we prefer to instead expandavatar's allowed types to let users pass in theRhUiAiChatbotIconthemselves? Answer - yes to default Avatar, also allowed other types.