Problem/Motivation

Leaflet More Maps uses old tile format strings for Thunderforest map styles. Besides, those old URLs do not support HTTPS, thus tiles will not even be displayed if the site uses TLS. We should switch to this new URL format, as it is described on the reference page for each style.

Finally, we have the opportunity to include newly created Thunderforest styles in the list.

Proposed resolution

The attached patch changes URL patterns, and adds new Thunderforest map styles.

Remaining tasks

Review patch.

User interface changes

Site builders will see new map styles appear in Leaflet field formatter settings.

API changes

None. Machine names of already existing maps will not change.

Data model changes

None.

Comments

FMB created an issue. See original summary.

fmb’s picture

Status: Active » Needs review
StatusFileSize
new4.6 KB
duaelfr’s picture

The code looks good to me.

rdeboer’s picture

Please provide patch for D7 as well...

Thanks
Rik

fmb’s picture

The patch for D7.

fmb’s picture

StatusFileSize
new4.97 KB
new4.22 KB

An updated patch for D8, with correct zooms in labels (these maps go up to zoom 22! I love this guy's work).

dawnbuie’s picture

The #5 D7 patch seems to work fro me. Thanks.

fmb’s picture

Status: Needs review » Reviewed & tested by the community

Taking the liberty to RTBC since this has been reviewed twice and reported to work.

japerry’s picture

Version: 8.x-1.x-dev » 7.x-1.x-dev
Status: Reviewed & tested by the community » Patch (to be ported)

D8 version committed. Setting to D7.

  • japerry committed dc5cf81 on 8.x-1.x authored by FMB
    Issue #2848895 by FMB: Update Thunderforest maps
    

  • berramou committed 5f567a70 on 7.x-1.x
    Issue #2848895 by fmb: Update Thunderforest maps
    
berramou’s picture

Status: Patch (to be ported) » Fixed
berramou’s picture

Status: Fixed » Closed (fixed)