Skip to content
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

[Bug] Heapmap: Tooltip+crosshair broken on Firefox #12486

Closed
grobie opened this issue Jul 2, 2018 · 0 comments · Fixed by #12999
Closed

[Bug] Heapmap: Tooltip+crosshair broken on Firefox #12486

grobie opened this issue Jul 2, 2018 · 0 comments · Fixed by #12999
Assignees
@alexanderzobnin @Ijin08
Labels
area/panel/heatmap type/bug
Milestone
5.3.0-beta1

Comments

@grobie
Copy link

grobie commented Jul 2, 2018

What Grafana version are you using?

v5.2.1

What datasource are you using?

Prometheus

What OS are you running grafana on?

Verified to be broken on Firefox on Linux and OSX. Works in Chrome on OSX

What did you do?

Create a heatmap graph

What was the expected result?

Hovering at any point over the heatmap shows the details of the underlying bucket

What happened instead?

Tooltip and crosshair only work on the very first pixel of a bucket, at all other pixels the crosshair jumps to T0 of that graph and no tooltip is visible.
@marefr marefr changed the title Heatmap: Tooltip+crosshair broken on Firefox [Bug] Heapmap: Tooltip+crosshair broken on Firefox Jul 4, 2018
@marefr marefr added this to the 5.3 milestone Jul 4, 2018
@marefr marefr assigned marefr and Ijin08 and unassigned marefr Aug 13, 2018
@alexanderzobnin alexanderzobnin self-assigned this Aug 22, 2018
@alexanderzobnin alexanderzobnin mentioned this issue Aug 22, 2018
Merged
@marefr marefr mentioned this issue Aug 23, 2018
Closed
marefr added a commit that referenced this issue Aug 23, 2018
@marefr
changelog: add notes about closing #12486
a9497f0 
[skip ci]
ryantxu added a commit to ryantxu/grafana that referenced this issue Aug 29, 2018
@ryantxu
Merge remote-tracking branch 'grafana/master'
8cfad74 
* grafana/master: (52 commits)
  Moved tooltip icon from input to label grafana#12945 (grafana#13059)
  added empty cta to playlist page + hid playlist table when empty (grafana#12841)
  Update provisioning.md
  changelog: add notes about closing grafana#12865
  fixed so validation of empty fields works again
  tslint: tslint to const fixes part3 (grafana#13036)
  tslint: more const fixes (grafana#13035)
  tslint: changing vars -> const (grafana#13034)
  tslint: autofix of let -> const (grafana#13033)
  WIP Update tslint (grafana#12922)
  changelog: add notes about closing grafana#12952 grafana#12965
  build: fixes rpm build when using defaults.
  changelog: add notes about closing grafana#12486
  docs: changes
  fixed so animation starts as soon as one pushes the button and animation stops if login failed
  added link to getting started to all, changed wording
  tests: fix missing tests (with .jest suffix)
  heatmap: fix tooltip bug in firefox
  Update notifications.md
  sql: added code migration type
  ...
ryantxu added a commit to ryantxu/grafana that referenced this issue Aug 29, 2018
@ryantxu
Merge remote-tracking branch 'grafana/master' into panel-visibility-flag
df8b9fc 
* grafana/master: (52 commits)
  Moved tooltip icon from input to label grafana#12945 (grafana#13059)
  added empty cta to playlist page + hid playlist table when empty (grafana#12841)
  Update provisioning.md
  changelog: add notes about closing grafana#12865
  fixed so validation of empty fields works again
  tslint: tslint to const fixes part3 (grafana#13036)
  tslint: more const fixes (grafana#13035)
  tslint: changing vars -> const (grafana#13034)
  tslint: autofix of let -> const (grafana#13033)
  WIP Update tslint (grafana#12922)
  changelog: add notes about closing grafana#12952 grafana#12965
  build: fixes rpm build when using defaults.
  changelog: add notes about closing grafana#12486
  docs: changes
  fixed so animation starts as soon as one pushes the button and animation stops if login failed
  added link to getting started to all, changed wording
  tests: fix missing tests (with .jest suffix)
  heatmap: fix tooltip bug in firefox
  Update notifications.md
  sql: added code migration type
  ...
ryantxu added a commit to ryantxu/grafana that referenced this issue Aug 29, 2018
@ryantxu
Merge remote-tracking branch 'grafana/master' into lazy-load-panels
4b7d870 
* grafana/master: (52 commits)
  Moved tooltip icon from input to label grafana#12945 (grafana#13059)
  added empty cta to playlist page + hid playlist table when empty (grafana#12841)
  Update provisioning.md
  changelog: add notes about closing grafana#12865
  fixed so validation of empty fields works again
  tslint: tslint to const fixes part3 (grafana#13036)
  tslint: more const fixes (grafana#13035)
  tslint: changing vars -> const (grafana#13034)
  tslint: autofix of let -> const (grafana#13033)
  WIP Update tslint (grafana#12922)
  changelog: add notes about closing grafana#12952 grafana#12965
  build: fixes rpm build when using defaults.
  changelog: add notes about closing grafana#12486
  docs: changes
  fixed so animation starts as soon as one pushes the button and animation stops if login failed
  added link to getting started to all, changed wording
  tests: fix missing tests (with .jest suffix)
  heatmap: fix tooltip bug in firefox
  Update notifications.md
  sql: added code migration type
  ...
ryantxu added a commit to ryantxu/grafana that referenced this issue Aug 29, 2018
@ryantxu
Merge remote-tracking branch 'grafana/master' into timepicker-tooltip
01acba4 
* grafana/master: (52 commits)
  Moved tooltip icon from input to label grafana#12945 (grafana#13059)
  added empty cta to playlist page + hid playlist table when empty (grafana#12841)
  Update provisioning.md
  changelog: add notes about closing grafana#12865
  fixed so validation of empty fields works again
  tslint: tslint to const fixes part3 (grafana#13036)
  tslint: more const fixes (grafana#13035)
  tslint: changing vars -> const (grafana#13034)
  tslint: autofix of let -> const (grafana#13033)
  WIP Update tslint (grafana#12922)
  changelog: add notes about closing grafana#12952 grafana#12965
  build: fixes rpm build when using defaults.
  changelog: add notes about closing grafana#12486
  docs: changes
  fixed so animation starts as soon as one pushes the button and animation stops if login failed
  added link to getting started to all, changed wording
  tests: fix missing tests (with .jest suffix)
  heatmap: fix tooltip bug in firefox
  Update notifications.md
  sql: added code migration type
  ...
@marefr marefr modified the milestones: 5.3, 5.3.0-beta1 Sep 6, 2018
This was referenced Sep 11, 2018
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@alexanderzobnin alexanderzobnin

@Ijin08 Ijin08

Labels
area/panel/heatmap type/bug
Projects
None yet
Milestone
5.3.0-beta1
Development

Successfully merging a pull request may close this issue.

Heatmap: fix tooltip and crosshair in firefox alexanderzobnin/grafana
4 participants
@grobie @marefr @alexanderzobnin @Ijin08