Demo page only works partially, many maps don't show

Created on 14 October 2023, over 1 year ago

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 📌 Stamen map tiles moving to Stadia hosting and will no longer be available at old URL Needs review ):

  • 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

🐛 Bug report
Status

Active

Version

2.1

Component

Code

Created by

🇩🇰Denmark ressa Copenhagen

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

  • Issue created by @ressa
  • 🇩🇰Denmark ressa Copenhagen
  • 🇫🇷France berramou

    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

  • 🇩🇰Denmark ressa Copenhagen

    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?

  • 🇫🇷France berramou

    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.

Production build 0.71.5 2024