Skip to content

feat(Message): add default bot message avatar#850

Open
kmcfaul wants to merge 10 commits into
patternfly:mainfrom
kmcfaul:chatbot-bot-icon
Open

feat(Message): add default bot message avatar#850
kmcfaul wants to merge 10 commits into
patternfly:mainfrom
kmcfaul:chatbot-bot-icon

Conversation

@kmcfaul

@kmcfaul kmcfaul commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Closes: #816

Todos -

  • pull in new prerelease react versions for Avatar update testing
  • update examples: remove avatar src prop from all bot message examples
  • update any tests
  • pull in new stable versions for Avatar update (6.6)

Open question -

  • Do we want to try to set the bot avatar by default like I currently have the logic in Message or would we prefer to instead expand avatar's allowed types to let users pass in the RhUiAiChatbotIcon themselves? Answer - yes to default Avatar, also allowed other types.

@patternfly-build

patternfly-build commented Jun 23, 2026

Copy link
Copy Markdown

Comment thread packages/module/src/Message/Message.tsx Outdated

@thatblindgeye thatblindgeye left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Comment thread packages/module/src/Message/Message.tsx
Comment thread packages/module/src/Message/Message.tsx Outdated
@kmcfaul kmcfaul force-pushed the chatbot-bot-icon branch from 58b265c to b69ea93 Compare June 25, 2026 14:30

@mcoker mcoker left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

    ---------------
    | bleep bloop |
    ---------------
   /
🤖

@thatblindgeye

Copy link
Copy Markdown
Collaborator

Now we just wait for official release numbers 😎

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update bot avatar icon

6 participants