Skip to content

scattergeo hoverinfo breaks when it's an array #2044

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
cpsievert opened this issue Sep 29, 2017 · 4 comments · Fixed by #2055
Closed

scattergeo hoverinfo breaks when it's an array #2044

cpsievert opened this issue Sep 29, 2017 · 4 comments · Fixed by #2055
Assignees
@etpinard
Labels
bug something broken

Comments

@cpsievert
Copy link

https://codepen.io/cpsievert/pen/NagamG

Discovered via plotly/plotly.R#1127
@cpsievert cpsievert mentioned this issue Sep 29, 2017
Closed
@etpinard etpinard added the bug something broken label Sep 29, 2017
@etpinard etpinard self-assigned this Sep 29, 2017
@etpinard
Copy link
Contributor

Should be an easy fix. I'll try to include in 1.31.0.

@etpinard
Copy link
Contributor

Oh wait, you have hoverinfo as an array (not hovertext like a I was expected - I must be going blind 👓 ). That's not supposed to work.

hoverinfo is flaglist attribute. Setting it to "lon+lat" gives the desired behavior:

https://codepen.io/etpinard/pen/veZRLw?editors=0010

@cpsievert
Copy link
Author

Why, then, would arrayOk be true?

screen shot 2017-09-29 at 1 57 43 pm

@etpinard
Copy link
Contributor

Oh sorry, I'm not going blind, but I might have Alzheimer's disease.

I forgot about #1761 - which didn't bother testing any geo traces. On it.

etpinard added a commit that referenced this issue Oct 3, 2017
@etpinard
fixes #2044 - add hoverinfo arrayOk support to ...
0a77239 
... scattergeo, choropleth, scatterternary and scattermapbox

- these are all the trace type that rely on `extraText` built in
  their _module.hoverPoints method
- this commit should've been part of PR #1761
@etpinard etpinard mentioned this issue Oct 3, 2017
Merged
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.

ArrayOk hoverinfo fixups plotly/plotly.js
2 participants
@cpsievert @etpinard