Voxel Probe domElement position

The positioning of the element containing voxel data only works when the renderer’s parent element is positioned at the top-left corner of the document. If the the renderer parent is contained in another element and its positioning with respect to the document is not at the top-left corner you should take in account its offset.

I made a little patch to the “updateRaycaster” function but I’m not sure if it’s the correct way to address the problem, neither if it conforms to your code conventions…

Anyway, here’s my patch:

updateRaycaster(raycaster, event, container) {
  // calculate mouse position in normalized device coordinates
  // (-1 to +1) for both components
  
  var offsets = this.getContainerOffsets();

  this._mouse = {
    x: (event.clientX - offsets.left) / container.offsetWidth * 2 - 1,
    y: -((event.clientY - offsets.top) / container.offsetHeight) * 2 + 1,
    screenX: event.clientX - offsets.left,
    screenY: event.clientY - offsets.top
  };

  // update the raycaster
  raycaster.setFromCamera(this._mouse, this._camera);
}

getContainerOffsets() { // crossbrowser version
  //Taken from http://stackoverflow.com/a/26230989/6172502
  var box = container.getBoundingClientRect();

  var body = document.body;
  var docEl = document.documentElement;

  var scrollTop = window.pageYOffset || docEl.scrollTop || body.scrollTop;
  var scrollLeft = window.pageXOffset || docEl.scrollLeft || body.scrollLeft;

  var clientTop = docEl.clientTop || body.clientTop || 0;
  var clientLeft = docEl.clientLeft || body.clientLeft || 0;

  var top  = box.top +  scrollTop - clientTop;
  var left = box.left + scrollLeft - clientLeft;

  return { top: Math.round(top), left: Math.round(left) };
}

[okrad]

Ok, I’m working on it… Expect a pull request on a couple of days…

[NicolasRannou]

Thanks @okrad that would be very useful!