predixdesignsystem / px-vis

A library of visualization components
http://predixdev.github.io/predix-ui/?show=px-vis&type=component
Apache License 2.0
34 stars 46 forks source link

px-vis-cursor/d3 attribute errors #123

Closed kjrdel closed 6 years ago

kjrdel commented 6 years ago

We are seeing Javascript errors stemming from px-vis-cursor when hovering over data on a time series plot where the coordinates are zero or undefined:

cursor-errors

We realize that this issue has been fixed in latest version 4 of px-vis:

https://github.com/PredixDev/px-vis/commit/

However, our application was built using an older version of the Predix App Starter and we are therefore dependent upon an older version of px-theme and px-vis (currently using px-vis version 2.1.9).

We were wondering if it might be possible to get this fix propagated down into version 2 until we are able to update our application to use the latest px-theme, designs, and visualization framework. The errors are having an impact on our app's performance.

Thanks!

nonmetalhail commented 6 years ago

Sorry, we cannot support legacy versions.

You can turn off the cursor lines and circle to prevent the console errors, but with the tradeoff that you wont have any indicator of where you are hovering over.