Replace `@plotly/mapbox-gl` with `mapbox-gl`

[MichaelDeBoey]

@plotly/mapbox-gl is a fork of mapbox-gl "maintained" by the @plotly team (@archmoj & @gvwilson) and used in plotly.js.
However, their version is still on v1.3, while upstream is already on v3.

In the light of getting rid of is-builtin-module (see #228), we should try to replace @plotly/mapbox-gl with mapbox-gl, since mapbox-gl already got rid of is-builtin-module in mapbox/mapbox-gl-js@0e3c67d

---

npx github:Fuzzyma/e18e-tools @plotly/mapbox-gl -n 100 -q -o md  -U https://npm.devminer.xyz/registry

#	Downloads	Traffic	Version	Package	PR	Issue
1	1.14M	985.38 GB	1.13.4	plotly.js		plotly/plotly.js#7174
2	9	7.79 MB	1.13.4	@kausal/plotly-custom

[gvwilson]

neither archmoj nor I are with Plotly any longer.

[MichaelDeBoey]

It seems @alexshoe @camdecoster & @emilykl are now the main contributors of Plotly

[archmoj]

Hi @MichaelDeBoey,
Thanks for opening the issue with the details.
Please note that mapbox subplots is now deprecated in plotly.js v3 and the users are encouraged to use map subplot instead which is base on maplibre-gl.
FYI - mapbox suplots would be removed in plotly.js v4 soon. See #7174.
So IMHO we could close this issue.

[MichaelDeBoey]

Let's keep keep this one open until plotly/plotly.js#7174 is done

[43081j]

@MichaelDeBoey interestingly i wasn't aware of mapliibre-gl too, maybe some effort could be made to move to that rather than mapbox-gl, if the latter is commercially licensed.

just a thought. if it is smaller (looks like it is), and more modern, it may be a good move

[MichaelDeBoey]

@43081j I haven't looked into the differences yet, but if maplibre-gl is a drop-in replacement for mapbox, I think it would indeed definitely make sense to point people towards the former

That still would require work on maplibre-gl to fix #228 though, since that one is fixed in mapbox, but not (yet) in maplibre-gl