Automattic / wp-calypso

The JavaScript and API powered WordPress.com
https://developer.wordpress.com
GNU General Public License v2.0
12.43k stars 1.99k forks source link

Map block displays markers as bullet points on Simple sites #93872

Open marissaexplores opened 2 months ago

marissaexplores commented 2 months ago

Quick summary

The map block breaks on Simple sites when adding more than 37 markers.

Map block with 37 markers: map-block-37-markers

Map block with 38 markers: map-block-38-markers

Steps to reproduce

  1. Add a map block to any page on a simple site
  2. Add 38 markers to the map
  3. View the bulleted list of markers on the frontend

What you expected to happen

The map should display a map with all the markers.

What actually happened

After adding the 38th marker to the map, the frontend displays a bulleted list of markers.

Impact

Some (< 50%)

Available workarounds?

No but the platform is still usable

If the above answer is "Yes...", outline the workaround.

If a user needs to add more than 37 markers to the map, they will need to use multiple map blocks. I don't think users will do this, so I'm marking this bug as no workaround

I ran into this issue when building a DIFM express site for a customer. We chose to not include multiple maps and instead listed the locations on the page with links to Google maps for each location.

Platform (Simple and/or Atomic)

Simple

Logs or notes

I tested the map on Assembler and Twenty Twenty-Four on simple sites, as well as testing both themes on an atomic site. I was able to add 43 markers to the map on the atomic site without problem each time. The issue occurred on both themes on the simple sites.

agrullon95 commented 2 months ago

@marissaexplores - Can you link the blog where youre seeing the issue? I have tried adding more than 38 markers to a map block on my Simple site (Twenty Twenty-Four theme) and Im able to see the map on the frontend.

marissaexplores commented 2 months ago

Sure! Here is an example. The top map is 37 and the bottom is 38.