Skip to content

Remove visual zone boundary overlays from Bulawayo map#74

Merged
roy-bme merged 1 commit into
claude/blackmass-corporate-website-caGTRfrom
codex/remove-zone-overlay-rectangles-from-map
May 14, 2026
Merged

Remove visual zone boundary overlays from Bulawayo map#74
roy-bme merged 1 commit into
claude/blackmass-corporate-website-caGTRfrom
codex/remove-zone-overlay-rectangles-from-map

Conversation

@roy-bme
Copy link
Copy Markdown
Owner

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

Motivation

  • The zone boundary rectangles/polygons were visually cluttering the map and obscuring business pins, so the map should no longer render filled zone overlays while keeping zone names and zone data intact.

Description

  • Removed the rendered zone overlay layer by deleting the dedicated zones LayerGroup and its management in components/ops/Map/BulawayoMap.tsx.
  • Removed zonesPane creation and the L.geoJSON(...)-based polygon rendering and hover styling so no filled rectangles/polygons are added to the map.
  • Removed the now-unused ZONE_TYPE_COLORS and zoneColor helpers and cleaned up the layersRef type to exclude zones while preserving zoneLabels and other layers.
  • Kept zone label markers (via zoneLabelsPane / zoneLabels layer) and preserved zones data usage such as zoneNameById and zone_id mapping for filtering/assignment logic.

Testing

  • Ran the TypeScript type check with npm run typecheck (which runs tsc --noEmit) and it completed successfully.

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 5:20pm

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 11eedc0 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