the-graph icon indicating copy to clipboard operation
the-graph copied to clipboard

[bug] Free end of a new edge position

Open YanDoroshenko opened this issue 7 years ago • 4 comments

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

YanDoroshenko avatar Aug 09 '17 20:08 YanDoroshenko

Which browser was this on?

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

jonnor avatar Aug 17 '17 14:08 jonnor

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

YanDoroshenko avatar Aug 17 '17 15:08 YanDoroshenko

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.

jonnor avatar Aug 23 '17 09:08 jonnor

I am seeing this with 0.11.1 as well.

KMontag42 avatar Jan 02 '18 21:01 KMontag42