You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Thanks so much for the report and for tracking this down! That makes pretty good sense. Would you be willing to make a PR for this? This might be small and straightforward enough that it doesn't require a specific test (debatably), but it's still necessary to ensure the tests as a whole are fine. The basic process is:
fork the repo
make the fix
run npm run lint-fix, commit, and push to your branch
file a PR
That would be greatly appreciated but no pressure. If not I'll try to get a PR together for it. Thanks again!
inDream
added a commit
to inDream/plotly.js
that referenced
this issue
Jan 26, 2017
Change https://github.com/plotly/plotly.js/blob/master/src/components/drawing/index.js#L54
from
if(isNumeric(x) && isNumeric(y)) {
toif(isNumeric(x) && isNumeric(y) && sel.node()) {
solved this problem.
Demo: https://jsfiddle.net/indream/6L8nLgr1/
The text was updated successfully, but these errors were encountered: