search

plotly / plotly.js

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

mapbox attribute control toggle, wrong styling #7082

Open birkskyum opened 2 months ago

birkskyum commented 2 months ago

mapbox plots like mapbox_scattercluster appear to have issues with the little attribution icon in the lower right

Present here: https://plotly.com/python/mapbox-layers/

Present in older builds, before merging maplibre. I checked out an earlier commit, and it seems like this appear before the MapLibre merger as well locally.

Mapbox:

Screenshot 2024-08-09 at 16 04 24 Screenshot 2024-08-09 at 16 08 35 Screenshot 2024-08-09 at 16 08 30 Screenshot 2024-08-09 at 12 40 53 Screenshot 2024-08-09 at 12 42 06

MapLibre:

Screenshot 2024-08-09 at 12 41 59 Screenshot 2024-08-09 at 12 42 33

Related

birkskyum commented 2 months ago

Using the codepen from here i find this issue is Present from plotly.js version 2.28.0 and forward.

2.28:

Screenshot 2024-08-09 at 16 39 19 Screenshot 2024-08-09 at 16 41 21

2.27:

Screenshot 2024-08-09 at 16 39 12 Screenshot 2024-08-09 at 16 41 51
archmoj commented 2 months ago

Likely related to #6782.

birkskyum commented 2 months ago

Using the .css from mapbox-gl 1.13.3 looks like this:

Screenshot 2024-08-09 at 17 11 58 Screenshot 2024-08-09 at 17 11 54
archmoj commented 2 months ago

@ndrezn Should one try to fix this new bug in mapbox?

ndrezn commented 2 months ago

I think this is low priority and we might not need to fix at all as we're deprecating these traces and the MapLibre traces will be unaffected.