Skip to content

Remove bot icon from AI Tutor and AI Chat Lab assistant messages#71738

Merged
fisher-alice merged 2 commits intostagingfrom
alice/remove-bot-chat-msg
Mar 27, 2026
Merged

Remove bot icon from AI Tutor and AI Chat Lab assistant messages#71738
fisher-alice merged 2 commits intostagingfrom
alice/remove-bot-chat-msg

Conversation

@fisher-alice
Copy link
Copy Markdown
Contributor

@fisher-alice fisher-alice commented Mar 27, 2026

This PR removes the AI Both icon from assistant messages in AI Tutor, AI Chat lab, but NOT AI TA. See Slack thread. This will allow more space in the chat workspace for content.

Links

Testing story

Tested locally in AI Tutor in Web Lab 2, AI Chat Lab, and confirmed no change in AI TA.

Before update

Assistant message in Web Lab 2 (with bot icon):

weblab2-before

Assistant message in AI Chat Lab (with bot icon):

aichat-before

After update

Assistant message in Web Lab 2 (no bot icon):

weblab2-after

Assistant message in AI Chat Lab (no bot icon):

aichat-after

AI TA (no change):

aita-after-no-change

Deployment notes

Privacy and security

@fisher-alice fisher-alice marked this pull request as ready for review March 27, 2026 21:08
@fisher-alice fisher-alice requested a review from a team March 27, 2026 21:09
)}
>
{isAssistant && isTA && (
<div className={moduleStyles.botIconWrapper}>
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.

It takes a little mental gymnastics to understand the difference between botIconWrapper and botIconContainer now - I wonder if it would be clearer to rename botIconWrapper to botIconOverlayContainer or botIconTAOverlayContainer?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Sure! yeah it took me a bit to understand this layout. I'll rename to botIconTAOverlayContainer - thanks! I think Tess may end up requesting that the bot icon get removed here also which would let us simplify.

Copy link
Copy Markdown
Contributor

@cnbrenci cnbrenci left a comment

Choose a reason for hiding this comment

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

One naming nit about wrapper vs container but otherwise lgtm!

@fisher-alice fisher-alice merged commit 7b290cb into staging Mar 27, 2026
8 checks passed
@fisher-alice fisher-alice deleted the alice/remove-bot-chat-msg branch March 27, 2026 22:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants