Merged
Conversation
zmanian
force-pushed
the
pr/attachment-ux-polish
branch
2 times, most recently
from
283bee9 to
bd28903
Compare
mcintyre94
force-pushed
the
pr/attachment-ux-polish
branch
from
bd28903 to
6c299b7
Compare
zmanian
force-pushed
the
pr/attachment-ux-polish
branch
from
6c299b7 to
94897d3
Compare
Owner
|
This is an awesome UX improvement, thankyou! Love this, great idea! |
- Replace confirmationDialog with inline Menu on attachment button - Add AttachedFile model and attachment chips view instead of raw paths - Show removable capsule chips above input bar for pending attachments - Add upload success feedback with auto-clearing filename indicator - Convert SpriteFileBrowserView from binding to callback pattern - Change file browser loading label to "Connecting to sprite..." - Fix duplicate PendingUserBubbleView entry in project file Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Store queued text and attachments separately so the pending bubble shows nice attachment chips instead of raw file paths. Defer prompt building to send time via buildPrompt(). Make AttachmentChipsView's remove button optional so it can be reused read-only in the pending bubble. For image attachments, the built prompt now includes a hint telling Claude Code to use its Read tool to view the image, enabling multimodal support without native CLI image input. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
mcintyre94
force-pushed
the
pr/attachment-ux-polish
branch
from
29b2bc5 to
23633a6
Compare
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
2 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
confirmationDialogwith inlineMenuon the attachment button for faster, less disruptive interactionSpriteFileBrowserViewfrom@Binding selectedPathtoonFileSelectedcallbackPendingUserBubbleViewentry in project fileTest plan
🤖 Generated with Claude Code