BUG: CanvasWrapper's onDrop event assumes canvas is positioned in the top left of the document.

[mrblenny]

BUG: CanvasWrapper's onDrop event assumes canvas is positioned in the top left of the document.

[phlickey]

On further testing, offsetLeft and offsetTop probably aren't sufficient.

Would probably need to be something like:

let elementOffset;
  if (this.ref.current){
    let {x,y} = this.ref.getBoundingClientRect()
    elementOffset = { x, y }
  }else{
    elementOffset = {x: 0, y: 0}
  }

Is there a better place that this offset could be calculated and stored somewhere stateful? Does it make sense to add it to the resize observer?

[mayansalama]

This has been bugging me as well - would be great to get this sorted.

[phlickey]

Looks like the offset is already being stored in state, and updated through the resize observer, just not being used when calculating the position property of the new node created on the onDrop event.

Made a PR that adds it in, since it seems pretty cut and dry and unlikely to affect anything else
#34