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] Free end of a new edge position #361

Open
YanDoroshenko opened this issue Aug 9, 2017 · 4 comments
Open

[bug] Free end of a new edge position #361

YanDoroshenko opened this issue Aug 9, 2017 · 4 comments
Labels

Comments

@YanDoroshenko
Copy link

After clicking a port a new edge is being created with the free end (that is supposed to be at the cursor position) in the random location. An error is being logged to console:

react.js:1768 Error: <g> attribute transform: Expected number, "translate(NaN,NaN)".
setValueForProperty @ react.js:1768
DOMPropertyOperations_setValueForProperty @ react.js:12353
_updateDOMProperties @ react.js:7042
updateComponent @ react.js:6923
receiveComponent @ react.js:6869
receiveComponent @ react.js:13028
_updateRenderedComponent @ react.js:5916
_performComponentUpdate @ react.js:5898
updateComponent @ react.js:5827
ReactCompositeComponent_updateComponent @ react.js:12353
receiveComponent @ react.js:5759
receiveComponent @ react.js:13028
updateChildren @ react.js:4143
_reconcilerUpdateChildren @ react.js:11667
_updateChildren @ react.js:11802
updateChildren @ react.js:11777
_updateDOMChildren @ react.js:7095
updateComponent @ react.js:6924
receiveComponent @ react.js:6869
receiveComponent @ react.js:13028
updateChildren @ react.js:4143
_reconcilerUpdateChildren @ react.js:11667
_updateChildren @ react.js:11802
updateChildren @ react.js:11777
_updateDOMChildren @ react.js:7095
updateComponent @ react.js:6924
receiveComponent @ react.js:6869
receiveComponent @ react.js:13028
_updateRenderedComponent @ react.js:5916
_performComponentUpdate @ react.js:5898
updateComponent @ react.js:5827
ReactCompositeComponent_updateComponent @ react.js:12353
performUpdateIfNecessary @ react.js:5775
performUpdateIfNecessary @ react.js:13043
runBatchedUpdates @ react.js:13741
perform @ react.js:15633
perform @ react.js:15633
perform @ react.js:13698
flushBatchedUpdates @ react.js:13759
ReactUpdates_flushBatchedUpdates @ react.js:12353
closeAll @ react.js:15699
perform @ react.js:15646
batchedUpdates @ react.js:8456
enqueueUpdate @ react.js:13788
enqueueUpdate @ react.js:13378
enqueueSetState @ react.js:13544
ReactComponent.setState @ react.js:5202
changeTooltip @ the-graph.js:11942
showTooltip @ the-graph.js:11574
ka @ hammer.js:2638
emit @ hammer.js:2638
emit @ hammer.js:2638
tryEmit @ hammer.js:2638
recognize @ hammer.js:2638
recognize @ hammer.js:2638
z @ hammer.js:2638
handler @ hammer.js:2638
domHandler @ hammer.js:2638
@jonnor jonnor added the bug label Aug 15, 2017
@jonnor
Copy link
Member

jonnor commented Aug 17, 2017

Which browser was this on?

Can you re-test with latest git master (e2d990134b) / 0.10.1 ?

@YanDoroshenko
Copy link
Author

YanDoroshenko commented Aug 17, 2017

Chromium 60.0.3112.101.
Firefox 55.0.2. Firefox does not log the error though.
Persists on 0.10.1 from NPM.
image
image

@jonnor
Copy link
Member

jonnor commented Aug 23, 2017

Can confirm. When starting an edge, it is shown towards the place it the previous edge was dropped at - until a new motion event comes in.

@KMontag42
Copy link

I am seeing this with 0.11.1 as well.

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

No branches or pull requests

3 participants