Repository navigation
fix(chat): align queue actions and keep attachments beside messages - #3781
Merged
Merged
Conversation
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
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
When queued messages have attachments, the badge sits at the far end of the content area instead of following the text. Recovery descriptions also change the row height and vertically shift its controls. Group previews and attachments on a dedicated message line, and align normal and recovery controls to that line with consistent widths and icon sizes.
Type and Areas
Type: UI/UX and bug fix
Areas: Web UI and shared composer components
Changes
ChatComposerQueueItemMessageslot. Short previews retain their natural width; long previews shrink while their attachment badge remains visible next to the text.Verification
pnpm run check:web— type-check, typography, appearance, theme color and visual contracts passednode --test design-system/packages/ui/tests/*.test.mjsafter package build — 352 passedpnpm run i18n:audit;pnpm --dir src/web-ui run icons:check; focused ESLint;git diff --check— passedpnpm run motion:audit— no new queue motion identifiedThe local review fixture was rebuilt from the current production component source, with its visualization-shell CSS removed to avoid button-style interference. Its dark/light and narrow layouts were inspected for debugging; fixture screenshots are not evidence of native WebView fidelity. No live SSH, Remote Control, Peer Device or Detached Dispatch scenario was exercised. The focused tests include simulated Peer and host recovery; transport and persisted shapes are unchanged.
Reviewer Notes
Local history and source review found that
127c69929already changed queue presentation and actions, but retained the badge outside the content and centered controls against the entire row. The reported local preview had an older component bundle; rebuilding it alone would not fix the remaining attachment placement in production.Checklist