MM-69160: Fix capitalization and text size in agent mention reminder - #916
MM-69160: Fix capitalization and text size in agent mention reminder#916mattermost-code wants to merge 2 commits into
Conversation
Capitalize "Click here to loop in" and match the reminder's font size (14px/20px) to regular post text. Co-authored-by: mattermost-code <mattermost-code@users.noreply.github.com>
Guards the capitalized loop-in link text and regular post font size. Co-authored-by: mattermost-code <mattermost-code@users.noreply.github.com>
|
Cursor Agent can help with this pull request. Just |
|
Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI (base), Organization UI (inherited) Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (5)
📝 WalkthroughWalkthroughThe agent mention reminder now uses capitalized link text and regular post typography. New component and end-to-end tests cover successful and failed loop-ins, fallback rendering, capitalization, and font sizing, with the E2E test assigned to CI shard 1. ChangesAgent mention reminder
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
Suggested labels: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
🤖 LLM Evaluation ResultsOpenAI
❌ Failed EvaluationsShow 7 failuresOPENAI1. TestReactEval/[openai]_react_cat_message
2. TestConversationMentionHandling/[openai]_conversation_from_attribution_long_thread.json
3. TestConversationMentionHandling/[openai]_conversation_from_attribution_long_thread.json
4. TestConversationMentionHandling/[openai]_conversation_from_attribution_long_thread.json
5. TestConversationMentionHandling/[openai]_conversation_from_attribution_long_thread.json
6. TestConversationMentionHandling/[openai]_conversation_from_attribution_long_thread.json
7. TestDirectMessageConversations/[openai]_bot_dm_tool_introspection
Anthropic
❌ Failed EvaluationsShow 7 failuresANTHROPIC1. TestReactEval/[anthropic]_react_cat_message
2. TestConversationMentionHandling/[anthropic]_conversation_from_attribution_long_thread.json
3. TestConversationMentionHandling/[anthropic]_conversation_from_attribution_long_thread.json
4. TestConversationMentionHandling/[anthropic]_conversation_from_attribution_long_thread.json
5. TestConversationMentionHandling/[anthropic]_conversation_from_attribution_long_thread.json
6. TestConversationMentionHandling/[anthropic]_conversation_from_attribution_long_thread.json
7. TestDirectMessageConversations/[anthropic]_bot_dm_tool_introspection
This comment was automatically generated by the eval CI pipeline. |
Summary
When a user replies in an agent thread without
@mentioning the bot, an ephemeral reminder is shown ("To respond to an agent you must @mention them. …"). This reminder had two visual issues reported in MM-69160:click here to loop in @…).13px/18px, slightly smaller than regular post text.This PR makes the smallest scoped fix in
webapp/src/components/agent_mention_reminder_post.tsx:Click here to loop in @….14px/20px(the same size used elsewhere for post-body help text).QA test steps
@<bot> Helloand wait for the bot's threaded reply.Ticket Link
Jira https://mattermost.atlassian.net/browse/MM-69160
Screenshots
Release Note
Summary by CodeRabbit
Bug Fixes
Tests