Skip to content

feat: inline image edit overlays on hero sections#392

Merged
williamluke4 merged 2 commits intopreviewfrom
williamluke4/image-edit-overlay
Apr 9, 2026
Merged

feat: inline image edit overlays on hero sections#392
williamluke4 merged 2 commits intopreviewfrom
williamluke4/image-edit-overlay

Conversation

@williamluke4
Copy link
Copy Markdown
Member

Summary

  • Add a reusable EditableImageOverlay component that renders a hover-to-reveal camera icon on images, triggering the existing file select / crop / upload flow
  • Voucher hero section now has inline edit overlays on both the banner image and the icon/avatar (owner-only, permission-gated)
  • Pool hero section now has an inline edit overlay on the banner image (owner-only, permission-gated)
  • Remove ImageUploadField entries from the voucher and pool settings forms since editing now happens directly on the hero

Test plan

  • Navigate to a voucher you own, hover over banner → camera icon appears → click → select file → crop → confirm → banner updates immediately
  • Same flow for the voucher icon/avatar
  • Navigate to a pool you own, hover over banner → camera icon appears → upload flow works
  • Navigate to a voucher/pool you don't own → no edit overlays visible
  • Open Settings tab on voucher/pool → image fields are gone, other fields still work
  • Test on mobile (touch to reveal overlay)

🤖 Generated with Claude Code

Add camera icon overlays on voucher banner, voucher icon, and pool
banner that trigger the upload flow directly from the hero section,
matching social media UX patterns. Remove image upload fields from
the settings forms since editing now happens inline.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown
Contributor

vercel bot commented Apr 9, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
sarafu-network Ready Ready Preview, Comment Apr 9, 2026 0:05am

Request Review

@williamluke4
Copy link
Copy Markdown
Member Author

@williamluke4 williamluke4 changed the base branch from main to preview April 9, 2026 11:54
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@williamluke4 williamluke4 merged commit 6207414 into preview Apr 9, 2026
4 of 6 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.

1 participant