plotly / plotly.js

Open-source JavaScript charting library behind Plotly and Dash
https://plotly.com/javascript/
MIT License
17.07k stars 1.87k forks source link

choropleth: when geojson is set, the geojson file from cdn.plot.ly is still loaded first #6767

Open LucaVazz opened 1 year ago

LucaVazz commented 1 year ago

When using type: choropleth and setting a geojson url, the geojosn file from cdn.plot.ly still is loaded first.
I'd expect that only the specified geojson file is loaded and used.

(This would be useful to e.g. implement a strict CSP on a site using plotly)


Example: https://codepen.io/lucavazz/pen/jOdrWgV
(baed on https://codepen.io/lucavazz/pen/poGbybw / #4267)

var data = [{
   // ...
   geojson: 'https://unpkg.com/sane-topojson@4.0.0/dist/world_110m.json' 
   // this url is an example, in practice this would point to a relative assets path
}]

image


Is this intended behavior / is some configuration missing in the example to only use the provided geojson or is this a bug in the implementation?

Thanks in advance :)

gvwilson commented 4 months ago

Hi - we are tidying up stale issues and PRs in Plotly's public repositories so that we can focus on things that are still important to our community. Since this one has been sitting for a while, I'm going to close it; if it is still a concern, please add a comment letting us know what recent version of our software you've checked it with so that I can reopen it and add it to our backlog. If you'd like to submit a PR, we'd be happy to prioritize a review, and if it's a request for tech support, please post in our community forum. Thank you - @gvwilson

LucaVazz commented 4 months ago

@gvwilson This issue is unfortunately still relevant, could you please re-open it?

gvwilson commented 4 months ago

Thanks @LucaVazz - I'll add it to our backlog, but I don't know when someone will be able to get to it :-(