Skip to content

fix broken map marker images on the members map#1421

Merged
danieltott merged 10 commits intoVirtual-Coffee:mainfrom
pedaars:fix/broken-map-marker-images
Oct 14, 2025
Merged

fix broken map marker images on the members map#1421
danieltott merged 10 commits intoVirtual-Coffee:mainfrom
pedaars:fix/broken-map-marker-images

Conversation

@pedaars
Copy link
Contributor

@pedaars pedaars commented Oct 11, 2025

Linked Issue

closes #1400

Description

fix broken map marker images on the members map

@meg-gutshall can you test this fix please?

Methodology

updated both the "react-leaflet": "^5.0.0" and "react-leaflet-cluster": "^3.1.1" packages to fix the MapContainer already initialized bug I was seeing reported here here

I am 95% sure the above is what was causing the issues

updated the way the fallback icon was imported to fix the broken image being rendered on the map

Code of Conduct

By submitting this pull request, you agree to follow our Code of Conduct

@pedaars pedaars requested a review from a team as a code owner October 11, 2025 18:39
@netlify
Copy link

netlify bot commented Oct 11, 2025

👷 Deploy Preview for virtual-coffee-io processing.

Name Link
🔨 Latest commit 6d601a5
🔍 Latest deploy log https://app.netlify.com/projects/virtual-coffee-io/deploys/68ee9ad80761a200078bb2c8

Copy link
Contributor

@meg-gutshall meg-gutshall left a comment

Choose a reason for hiding this comment

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

Hey @pedaars! Thanks for looking into this further.

We definitely want to keep the updates in the package.json file. Unfortunately, the changes in the MemberMap.tsx file didn't solve the issue.

Screenshot 2025-10-14 at 10 47 49 AM

Perhaps you can pair with another member who is visually seeing the bug? It's difficult to fix a bug you're not experiencing yourself!

@JoeKarow
Copy link
Member

To fix the broken images, I switched to use the base Icon class instead of Icon.Default in createCustomIcon(). Docs Section.

I believe Icon.Default is meant to be a placeholder class or something. At any rate - it's rendering images 😄

@danieltott danieltott self-assigned this Oct 14, 2025
Copy link
Member

@danieltott danieltott left a comment

Choose a reason for hiding this comment

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

Great work! I love a team effort 💙

@danieltott danieltott merged commit eab94db into Virtual-Coffee:main Oct 14, 2025
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug: Fix Member Map

4 participants