Merge branch 'feature/unsnapped-tokens' into develop

Changelog.md

@@ -1,3 +1,7 @@
+# 0.7.8
+More tweaks to how token origin and destination are set when dragging so that the token movement follows the position of the cloned dragged token. Revisits issue #30.
+Fix issue where token dragging cannot move to the adjacent space. Closes issue #32.
+
 # 0.7.7
 Allow GM to move tokens using the ruler regardless of obstacle. Closes issue #27.
 Cancel the ruler when canceling the drag. Closes issue #28.

scripts/BaseGrid.js

@@ -18,8 +18,10 @@ function _getRulerDestination(wrapped, ray, offset, token) {
   if ( canvas.controls.ruler._unsnap ) return ray.B.add(offset);
 
   // We are moving from the token center, so add back 1/2 width/height to offset.
-  offset.x += canvas.scene.dimensions.size * 0.5;
-  offset.y += canvas.scene.dimensions.size * 0.5;
+  if ( !canvas.controls.ruler._unsnappedOrigin ) {
+    offset.x += canvas.scene.dimensions.size * 0.5;
+    offset.y += canvas.scene.dimensions.size * 0.5;
+  }
   return wrapped(ray, offset, token);
 }
 

scripts/Ruler.js

@@ -1,6 +1,6 @@
 /* globals
 canvas,
-duplicate,
+CONST,
 game,
 ui
 */
@@ -26,6 +26,8 @@ import {
   modifiedMoveDistance
 } from "./segments.js";
 
+import { tokenIsSnapped } from "./util.js";
+
 /**
  * Modified Ruler
  * Measure elevation change at each waypoint and destination.
@@ -83,6 +85,7 @@ function toJSON(wrapper) {
   const obj = wrapper();
   obj._userElevationIncrements = this._userElevationIncrements;
   obj._unsnap = this._unsnap;
+  obj._unsnappedOrigin = this._unsnappedOrigin;
   return obj;
 }
 
@@ -96,6 +99,7 @@ function update(wrapper, data) {
   const triggerMeasure = this._userElevationIncrements !== data._userElevationIncrements;
   this._userElevationIncrements = data._userElevationIncrements;
   this._unsnap = data._unsnap;
+  this._unsnappedOrigin = data._unsnappedOrigin;
   wrapper(data);
 
   if ( triggerMeasure ) {
@@ -114,6 +118,14 @@ function _addWaypoint(wrapper, point) {
 
   // If shift was held, use the precise point.
   if ( this._unsnap ) this.waypoints.at(-1).copyFrom(point);
+  else if ( this.waypoints.length === 1 ) {
+    // Move the waypoint to find unsnapped token.
+    const oldWaypoint = duplicate(this.waypoints[0]);
+    this.waypoints[0].copyFrom(point);
+    const token = this._getMovementToken();
+    if ( token && !tokenIsSnapped(token) ) this._unsnappedOrigin = true;
+    else this.waypoints[0].copyFrom(oldWaypoint);
+  }
 
   // Elevate the waypoint.
   addWaypointElevationIncrements(this, point);
@@ -183,8 +195,6 @@ function _computeDistance(wrapped, gridSpaces) {
   for ( const segment of this.segments ) {
     segment.moveDistance = modifiedMoveDistance(segment.distance, segment.ray, token);
     totalMoveDistance += segment.moveDistance;
-    // const { A, B } = segment.ray;
-    //console.debug(`${A.x},${A.y},${A.z} --> ${B.x},${B.y},${B.z}: distance ${segment.distance}, move ${segment.moveDistance}`);
   }
   this.totalMoveDistance = totalMoveDistance;
 }

scripts/util.js

@@ -126,4 +126,14 @@ export function * iterateGridUnderLine(origin, destination, { reverse = false }
   }
 }
 
+/**
+ * Determine if a token is currently snapped to the grid.
+ * @param {Token} token
+ * @returns {boolean}
+ */
+export function tokenIsSnapped(token) {
+  const { x, y } = token.document;
+  const [snappedX, snappedY] = canvas.grid.grid.getTopLeft(x, y);
+  return snappedX.almostEqual(x) && snappedY.almostEqual(y);
+}