Skip to content

fix(web): preserve inline attachment image bounds - #6505

Closed
myNameArnav wants to merge 2 commits into
pingdotgg:mainfrom
myNameArnav:fix/inline-image-preview-crop
Closed

fix(web): preserve inline attachment image bounds#6505
myNameArnav wants to merge 2 commits into
pingdotgg:mainfrom
myNameArnav:fix/inline-image-preview-crop

Conversation

@myNameArnav

@myNameArnav myNameArnav commented Aug 13, 2026

Copy link
Copy Markdown

Problem

Inline user-image previews used object-cover, which cropped tall screenshots and cut into their rounded corners.

Fix

Use object-contain so the preview preserves the uploaded image bounds while the existing rounded, overflow-hidden wrapper continues to clip the corners cleanly.

Before / After

Before

After

Verification

  • vp run --filter @t3tools/web test
  • vp run --filter @t3tools/web typecheck
  • vp lint apps/web/src/components/chat/MessagesTimeline.tsx
  • Browser visual comparison with the supplied screenshot completed

Model: GPT-5.6-Luna | Harness: OpenCode in T3 Code

@coderabbitai

coderabbitai Bot commented Aug 13, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 77d32770-bb3c-44b1-971c-acef5c42c836

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions github-actions Bot added vouch:unvouched PR author is not yet trusted in the VOUCHED list. size:XS 0-9 changed lines (additions + deletions). labels Aug 13, 2026
@macroscopeapp

macroscopeapp Bot commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved 62f8a96

Single-line CSS change switching image scaling from object-cover to object-contain - a minor UI adjustment affecting only how inline attachment images are displayed without cropping.

You can customize Macroscope's approvability policy. Learn more.

@myNameArnav
myNameArnav marked this pull request as draft August 13, 2026 18:12
@myNameArnav

Copy link
Copy Markdown
Author

Closed after fresh browser reproduction showed the proposed object-fit change produced identical before/after rendering; no effective fix was shipped.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS 0-9 changed lines (additions + deletions). vouch:unvouched PR author is not yet trusted in the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant