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.

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

[Bug]: Heatmap Legend Tooltips incorrectly calculate Min value #1423

Closed
DakotaWray2 opened this issue Aug 5, 2022 · 1 comment · Fixed by #1434
Closed

[Bug]: Heatmap Legend Tooltips incorrectly calculate Min value #1423

DakotaWray2 opened this issue Aug 5, 2022 · 1 comment · Fixed by #1434
Assignees
Labels
type: bug 🐛 Something isn't working
Projects

Comments

@DakotaWray2
Copy link

Contact Details

Dakota.Wray2@gmail.com

What happened?

Heatmap chart tooltip min calculations default to zero even if no zero values in row/column. This can be observed in the storybook.

Version

latest

Data & options used

see storybook. Also observed in data passed into component.

Relevant log output

May not be related but I do get the following error when hovering over my x-axis. 

charts-vue.umd.min.js?0cc3:1 Uncaught TypeError: Cannot read properties of undefined (reading 'length')
    at e.getItems (charts-vue.umd.min.js?0cc3:1:1)
    at e.getTooltipHTML (charts-vue.umd.min.js?0cc3:1:1)
    at i.handleShowTooltip (charts-vue.umd.min.js?0cc3:1:1)
    at e.dispatchEvent (charts-vue.umd.min.js?0cc3:1:1)
    at SVGGElement.eval (charts-vue.umd.min.js?0cc3:1:1)
    at SVGGElement.eval (charts-vue.umd.min.js?0cc3:1:1)

There are no undefined values in the data passed into the component.

Codesandbox example

No response

@DakotaWray2 DakotaWray2 added the type: bug 🐛 Something isn't working label Aug 5, 2022
@theiliad
Copy link
Member

theiliad commented Aug 5, 2022

@Akshat55 could you pls look at this?

@Akshat55 Akshat55 added this to ToDo's in Q3- 2022 Aug 5, 2022
@Akshat55 Akshat55 moved this from ToDo's to In progress in Q3- 2022 Aug 10, 2022
Akshat55 added a commit to Akshat55/carbon-charts that referenced this issue Aug 22, 2022
@Akshat55 Akshat55 moved this from In progress to QA/Review in Q3- 2022 Aug 22, 2022
theiliad pushed a commit that referenced this issue Sep 14, 2022
@ninja511 ninja511 moved this from QA/Review to Closed in Q3- 2022 Sep 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug 🐛 Something isn't working
Projects
No open projects
Q3- 2022
Closed
3 participants