Skip to content

Fix business photo upload trigger, expose uploader in details panel, and raise panel z-index#77

Merged
roy-bme merged 1 commit into
claude/blackmass-corporate-website-caGTRfrom
codex/fix-multiple-photo-upload-panel-bugs
May 14, 2026
Merged

Fix business photo upload trigger, expose uploader in details panel, and raise panel z-index#77
roy-bme merged 1 commit into
claude/blackmass-corporate-website-caGTRfrom
codex/fix-multiple-photo-upload-panel-bugs

Conversation

@roy-bme
Copy link
Copy Markdown
Owner

@roy-bme roy-bme commented May 14, 2026

Motivation

  • Users could not trigger the photo file picker from the business side panel, mobile devices were forced into camera-only mode, and the edit/details panel rendered underneath the Leaflet map preventing interaction.
  • Photo upload functionality needed to be available from the main details view without forcing users into edit mode.

Description

  • Wired the visible "Add photo" button to the hidden file input by calling the file input ref in handleAddPhotoClick and added a debug console.log("file input clicked") before invoking click().
  • Removed the capture="environment" attribute from the file input so OS-level chooser offers camera or library when accept="image/*" is used.
  • Open the side panel when selecting a pin by setting selected in handleSelectFromPin and leave a non-editing details view available so photos can be added without entering edit mode; kept editing state for toggling edit UI.
  • Increased the side-panel and candidate-panel stacking context to z-[1000] so the panels render above Leaflet tiles/popups and fixed related className changes in components/ops/Map/MapView.tsx.

Testing

  • Ran npm run -s lint which completed successfully with only an informational Next.js warning about using <Image /> instead of <img>; no lint errors were reported.

Codex Task

@vercel
Copy link
Copy Markdown

vercel Bot commented May 14, 2026

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

Project Deployment Actions Updated (UTC)
blackmasswebsite Ready Ready Preview, Comment May 14, 2026 6:33pm

Request Review

@supabase
Copy link
Copy Markdown

supabase Bot commented May 14, 2026

This pull request has been ignored for the connected project ipqmdinidqpmchggjdov because there are no changes detected in supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.


Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

@roy-bme roy-bme merged commit 822881c into claude/blackmass-corporate-website-caGTR May 14, 2026
2 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant