Skip to content

Align sent attachment previews - #119

Closed
hamzamerzic wants to merge 1 commit into
mobius-os:mainfrom
hamzamerzic:fix/sent-attachment-preview-parity-20260722
Closed

Align sent attachment previews#119
hamzamerzic wants to merge 1 commit into
mobius-os:mainfrom
hamzamerzic:fix/sent-attachment-preview-parity-20260722

Conversation

@hamzamerzic

Copy link
Copy Markdown
Collaborator

Summary

  • render sent images as the same 96 by 96 square crop used by the composer
  • keep the full source available through the existing lightbox
  • align the focus boundary and image corners at the shared 14px radius
  • guard attachment-before-text ordering in both message render paths

Provenance

Squashes the production-local checkpoint 67ce0a58 and follow-ups e8a10051 plus 12343b57 into one final patch on current main 41a3809. The interim 96 by 192 variant is intentionally absent.

Validation

  • focused sizing and message-order tests: 2 passed
  • full reviewed UI stack: 1,765 frontend library tests and 47 hook tests passed
  • production frontend build passed

@hamzamerzic

Copy link
Copy Markdown
Collaborator Author

Two review rounds complete on exact head 5c22de243237f4d5decb4e1b7baef6b1cc467fdb. Pass 1 verified the 96×96 crop, shared 14px focus/image radius, and attachment-before-text ordering in both render paths. Pass 2 checked lightbox preservation, wrapping/responsive behavior, focus visibility, box sizing, and regression strength. Focused tests pass and all hosted checks are green. No finding; recommendation: merge.

@hamzamerzic

Copy link
Copy Markdown
Collaborator Author

Landed on protected main through reviewed integration #147 (merge 226106c). The integrated patch matches this PR's reviewed head; closing the superseded source PR to keep the queue truthful.

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.

1 participant