Problem/Motivation

Only some maps load on the Leaflet Demo page at /admin/config/system/leaflet-more-maps/demo.

Steps to reproduce

Visit the Demo page and see that some maps don't load. Many maps require an API key to work, so that they don't load makes sense. But also some free maps don't load.

On the Demo page, only these load, after patching Stamen maps (see #3393412: Update Stamen map tiles to use Stadia hosting):

  • Stamen Toner-Hybrid
  • Stamen Toner-Labels
  • Stamen Toner-Lines
  • Stamen Toner-Lite
  • Stamen Watercolor

Proposed resolution

Figure out why some free maps don't load, and make them work again.

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#3 Screen-more-maps-demo.mov14.61 MBberramou

Comments

ressa created an issue. See original summary.

ressa’s picture

Issue summary: View changes
berramou’s picture

StatusFileSize
new14.61 MB

I can't reproduce this bug with the last version on Drupal 11
Look at the screen recording, @ressa can you give more details and steps to reproduce please

ressa’s picture

Status: Active » Closed (cannot reproduce)

You're right @berramou, it must have been fixed, so I am updating the status, thanks for following up on this!

By the way, it's a little strange to see the only dev-branch is 2.1.x-dev, which has not been updated since June 2024, yet 2.2.3 was released January 2025 ...

  • 2.2.3 Stable released: 23 January 2025
  • 2.1.x-dev updated: 11 Jun 2024

Maybe you can become a co-maintainer, and tag a 2.2.x-dev branch, which can be ahead of 2.2.3, and become the dev-branch?

berramou’s picture

Yes i remember i have fixed it in another issue, it was caused by JS error that blocks some maps.
And about the branches, i'm a co-maintainer but i don't have right to branch settings.
The branch 2.2.x should be set as default branch.
I guess we need to ask other maintainer to do that.