Skip to content

heatmapgl cursor popup can't go wider than the plot is tall #1825

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.

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

Closed
ssfrr opened this issue Jun 28, 2017 · 3 comments · Fixed by #1884
Closed

heatmapgl cursor popup can't go wider than the plot is tall #1825

ssfrr opened this issue Jun 28, 2017 · 3 comments · Fixed by #1884
Labels
bug something broken

Comments

@ssfrr
Copy link

ssfrr commented Jun 28, 2017

I use heatmaps to plot spectrograms represented as matrices. In trying to improve performance I tried using heatmapgl, but it looks like there's an issue with the cursor popover where it stops following the cursor when the X coordinate is greater than 1023.

Example:
https://jsfiddle.net/xfj1yxyu/3/

EDIT: corrected fiddle URL

@etpinard etpinard added the bug something broken label Jun 28, 2017
@etpinard
Copy link
Contributor

cc @dfcreative

@ssfrr
Copy link
Author

ssfrr commented Jun 28, 2017

just realized that 1023 isn't special, but that's the height of my plot (well, 1025, but that's close enough). When the plot is taller than it is wide things seem to be OK.

A smaller (e.g. 50x100) heatmapgl shows the same behavior.

@ssfrr ssfrr changed the title heatmapgl cursor popup can't go past 1023 heatmapgl cursor popup can't go wide if the plot is tall Jun 28, 2017
@ssfrr ssfrr changed the title heatmapgl cursor popup can't go wide if the plot is tall heatmapgl cursor popup can't go wider than the plot is tall Jun 28, 2017
@dy
Copy link
Contributor

dy commented Jun 28, 2017

Same issue we have with contourgl type of plot.
Codepen.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug something broken
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants