Skip to content

t.split issue with choroplethmapbox on hover #4194

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
chriddyp opened this issue Sep 17, 2019 · 3 comments · Fixed by #4197
Closed

t.split issue with choroplethmapbox on hover #4194

chriddyp opened this issue Sep 17, 2019 · 3 comments · Fixed by #4197
Assignees
@etpinard
Labels
bug something broken

Comments

@chriddyp
Copy link
Member

It renders fine, but the hover info isn't appearing.

image

Traceback:
image
@chriddyp
Copy link
Member Author

chriddyp commented Sep 17, 2019 •
edited
Loading

plotly.js 1.49.4 on Firefox on mac

@etpinard
Copy link
Contributor

Reproducible codepen: https://codepen.io/etpinard/pen/pozOKjL?editors=1010

Thanks for reporting!

@etpinard etpinard added the bug something broken label Sep 17, 2019
@etpinard etpinard self-assigned this Sep 17, 2019
etpinard added a commit that referenced this issue Sep 17, 2019
@etpinard
fix #4194 - stringify locations[i] for hover pointData
ffb67f9 
- so the rest of Fx.hover can properly handle it,
  the bug currently only happens for number locations[i]
  that make it part of the <extra></extra> hover label content.
@etpinard etpinard mentioned this issue Sep 17, 2019
Merged
etpinard added a commit that referenced this issue Sep 18, 2019
@etpinard
fix #4194 - stringify locations[i] for hover pointData
6ae8a00 
- so the rest of Fx.hover can properly handle it,
  the bug currently only happens for number locations[i]
  that make it part of the <extra></extra> hover label content.
@chriddyp
Copy link
Member Author

thanks @etpinard! 🙌

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@etpinard etpinard

Labels
bug something broken
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

A few mapbox fixes plotly/plotly.js
2 participants
@chriddyp @etpinard