Redesign Chat tab UI and message interactions#2650
Draft
shaunandrews wants to merge 10 commits intotrunkfrom
Draft
Redesign Chat tab UI and message interactions#2650shaunandrews wants to merge 10 commits intotrunkfrom
shaunandrews wants to merge 10 commits intotrunkfrom
Conversation
Remove the sparkle icon CSS and ml-auto positioning that pushed the tab to the far right. Place it at order 0 so it appears as the first tab, inline with the rest. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Strip the useAiIcon hook, its typing/keyUp state machinery, and the Rive component from the input. Bump horizontal padding from px-1 to px-4 to fill the space left by the removed icon. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Strip the rounded border, bg-white, and bg-white/45 from normal messages for a cleaner look. Padding stays the same. Failed message styling (red border/background) is preserved. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Match the simplified styling from chat messages. Loading skeleton keeps its bg-gray-100 and rounded for the pulse placeholder effect. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Remove border and background from welcome messages to match chat - Add subtle background (a8c-gray-100) and rounded-xl to user messages - Override API welcome messages with Kit introduction copy Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Default color matches surrounding text (a8c-gray-70), colored on hover - Remove 'Was this helpful?' label, use tooltips instead - Rename Yes/No to Helpful/Unhelpful - Left-align buttons with message text, tighten spacing Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Copy button on all messages, visible on hover - Copy button on last assistant message (via LastMessage) - Replace 'Powered by experimental AI' with concise disclaimer - Center and shrink disclaimer text Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Create MessageActions bar (copy + ratings) shown on hover for all msgs - ChatRating is now a toggle: click selects, click again deselects - All assistant messages show ratings, not just the last one - Remove FeedbackThanks replacement text - Pass onRate callback from ContentTabAssistant through AuthenticatedView Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Restructure ChatMessage to wrap bubble + actions in inline-flex column so actions align to bubble width on both user and AI messages - Actions always left-aligned with pl-3 matching message padding - Remove bottom padding on AI messages (actions provide spacing) - Animated rating toggle with max-width transitions and filled icons - Fix rating button height alignment (flex wrapper vs block div) - Replace CopyTextButton with inline CopyButton for consistent sizing Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Collaborator
📊 Performance Test ResultsComparing 9aa3875 vs trunk site-editor
site-startup
Results are median values from multiple test runs. Legend: 🟢 Improvement (faster) | 🔴 Regression (slower) | ⚪ No change (<50ms diff) |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Test plan