Skip to content

feat(web): add lightbox preview for chat images#715

Merged
tiann merged 1 commit into
tiann:mainfrom
techotaku39:fix/web-chat-image-lightbox
May 27, 2026
Merged

feat(web): add lightbox preview for chat images#715
tiann merged 1 commit into
tiann:mainfrom
techotaku39:fix/web-chat-image-lightbox

Conversation

@techotaku39
Copy link
Copy Markdown
Contributor

Summary

  • extract the zoomable image viewer from the file page into a reusable ImagePreview component
  • use the same full-screen viewer for chat image attachments
  • make generated-image chat cards click-to-preview with the same zoom/pan controls

Reason

Chat images were only visible as small inline previews. Unlike the file viewer, clicking a chat image did not open a larger preview, which made screenshots and generated images hard to inspect on mobile.

This keeps the existing thumbnail layout, but lets users tap the image to open the full-screen viewer with zoom, pan, double-click reset, Escape close, and the existing file-viewer controls.

Related

Test

  • npm exec -- bun run typecheck:web

Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Findings

  • No high-confidence findings.

Summary

  • Review mode: initial
  • Reviewed the full latest diff for image attachment lightbox reuse across chat attachments, generated-image cards, and the file viewer. No correctness, security, data-loss, performance, or maintainability issue above the reporting threshold found in the added/modified lines.
  • Residual risk: web UI interaction coverage remains manual; no existing web tests cover the lightbox gestures or chat attachment preview flow.

Testing

  • Not run (automation): bun run typecheck:web could not run because bun is not available on PATH in this runner.

HAPI Bot

@tiann tiann merged commit cf378df into tiann:main May 27, 2026
2 checks passed
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.

2 participants