Add options as input argument to the getLeafletLayer() method

[oriolbx]

Context

According to the documentation, the methods getleafletLayer() , returns a leaflet layer that groups all the layers that have been added to this client, it returns a L.TileLayer layer that groups all the layers.

However, if you want to add or change the options of the layer, it is not possible using the methods of Leaflet and CARTO getLeafletLayer() method does not accepts input arguments to change the options of the Layer generated.

It would be nice to have this option in CARTO.js.

For example, at this moment we can add the option noWrap to the L.tileLayer of the basemap but not for CARTO layer. Therefore the result is the next one:

[Jesus89]

Hi @oriolbx.

This is a really good point in order to enable better integration with native Leaflet Layers.

This is also easy to implement:

• Add an options parameter to the getLeafletLayer() function
• Merge these options with our defaults
• Apply the merged options to the internal L.tileLayer instance

Thanks 👍

[rubenmoya]

I've opened a PR to solve this, although your example won't work if you only use noWrap, you can read more about it here. Using this options works:

{
  noWrap: true,
  bounds: [
    [-90, -180],
    [90, 180]
  ]
}

[rubenmoya]

Fixed in @carto/[email protected]