Skip to content

Render Indaba zone boundaries from GeoJSON polygons#70

Merged
roy-bme merged 1 commit into
claude/blackmass-corporate-website-caGTRfrom
codex/render-zone-boundaries-as-polygons
May 11, 2026
Merged

Render Indaba zone boundaries from GeoJSON polygons#70
roy-bme merged 1 commit into
claude/blackmass-corporate-website-caGTRfrom
codex/render-zone-boundaries-as-polygons

Conversation

@roy-bme
Copy link
Copy Markdown
Owner

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

Motivation

  • Zones were only shown as centre-point labels but the database now contains boundary_geojson polygons for all zones, so the map should display actual zone shapes.
  • Polygons should be visually differentiated by zone type, be semi-transparent, and render behind business pins so markers remain prominent.

Description

  • Updated the server-side fetch to include type and boundary_geojson from the zones table and pass them through to the client model (app/indaba/(ops)/map/page.tsx).
  • Extended MapZone to carry optional type and boundary_geojson fields (components/ops/Map/types.ts).
  • Implemented polygon rendering in the Leaflet map (components/ops/Map/BulawayoMap.tsx) using L.geoJSON(...) with zone-type color mapping, a semi-transparent fill (fillOpacity: 0.15), subtle border (opacity: 0.4), and hover interaction that increases fill opacity to 0.3 and shows the zone name tooltip.
  • Added Leaflet panes (zonesPane, zoneLabelsPane) and layer groups so polygons render behind business markers while keeping existing centre-point text labels.

Testing

  • Ran TypeScript checks with npm run typecheck, which completed successfully.
  • No additional automated tests were added or run.

Codex Task

@vercel
Copy link
Copy Markdown

vercel Bot commented May 11, 2026

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

Project Deployment Actions Updated (UTC)
blackmasswebsite Ready Ready Preview, Comment May 11, 2026 11:10am

Request Review

@supabase
Copy link
Copy Markdown

supabase Bot commented May 11, 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 23110c0 into claude/blackmass-corporate-website-caGTR May 11, 2026
3 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