Release v1.13.6

CHANGELOG.md

@@ -1,3 +1,9 @@
+## 1.13.6
+### Bugfixes
+- Fixed a bug that caused no measurements to be shown next to the ruler
+- Fixed a bug where diagonal paths would sometimes highlight squares that don't blong to the path on square maps
+
+
 ## 1.13.5
 ### Compatibility
 - Drag Ruler is now compatible with Foundry VTT v11 (thanks to pkonshik for doing much of the porting work!)

module.json

@@ -2,7 +2,7 @@
 	"id": "drag-ruler",
 	"title": "Drag Ruler",
 	"description": "When dragging a token displays a ruler showing how far you've moved that token.",
-	"version": "1.13.5",
+	"version": "1.13.6",
 	"compatibility": {
 		"minimum": "11",
 		"verified": "11"
@@ -80,7 +80,7 @@
 	},
 	"socket": true,
 	"url": "https://github.com/manuelVo/foundryvtt-drag-ruler",
-	"download": "https://github.com/manuelVo/foundryvtt-drag-ruler/archive/v1.13.5.zip",
+	"download": "https://github.com/manuelVo/foundryvtt-drag-ruler/archive/v1.13.6.zip",
 	"manifest": "https://raw.githubusercontent.com/manuelVo/foundryvtt-drag-ruler/master/module.json",
 	"readme": "https://github.com/manuelVo/foundryvtt-drag-ruler/blob/master/README.md",
 	"changelog": "https://github.com/manuelVo/foundryvtt-drag-ruler/blob/master/CHANGELOG.md",

src/foundry_imports.js

@@ -185,7 +185,7 @@ export function highlightMeasurementNative(
 	tokenShape = [{x: 0, y: 0}],
 	alpha = 1,
 ) {
-	const spacer = canvas.scene.gridType === CONST.GRID_TYPES.SQUARE ? 1.41 : 1;
+	const spacer = canvas.scene.grid.type === CONST.GRID_TYPES.SQUARE ? 1.41 : 1;
 	const nMax = Math.max(
 		Math.floor(ray.distance / (spacer * Math.min(canvas.grid.w, canvas.grid.h))),
 		1,

src/ruler.js

@@ -262,29 +262,23 @@ export function extendRuler() {
 			if (!this.isDragRuler) {
 				return super._computeDistance(gridSpaces);
 			}
-			if (!this.dragRulerEnableTerrainRuler) {
+			const shape = this.draggedEntity ? getTokenShape(this.draggedEntity) : null;
-				if (!this.dragRulerIgnoreGrid) {
+			const options = {
-					gridSpaces = true;
+				ignoreGrid: this.dragRulerIgnoreGrid,
-				}
+				gridSpaces,
-				super._computeDistance(gridSpaces);
+				enableTerrainRuler: this.dragRulerEnableTerrainRuler,
-			} else {
+			};
-				const shape = this.draggedEntity ? getTokenShape(this.draggedEntity) : null;
+			const distances = measureDistances(this.segments, this.draggedEntity, shape, options);
-				const options = {
+			let totalDistance = 0;
-					ignoreGrid: this.dragRulerIgnoreGrid,
+			for (const [i, d] of distances.entries()) {
-					gridSpaces,
+				let s = this.segments[i];
-					enableTerrainRuler: this.dragRulerEnableTerrainRuler,
+				s.startDistance = totalDistance;
-				};
+				totalDistance += d;
-				const distances = measureDistances(this.segments, this.draggedEntity, shape, options);
+				s.last = i === this.segments.length - 1;
-				let totalDistance = 0;
+				s.distance = d;
-				for (const [i, d] of distances.entries()) {
+				s.text = this._getSegmentLabel(s, totalDistance);
-					let s = this.segments[i];
-					s.startDistance = totalDistance;
-					totalDistance += d;
-					s.last = i === this.segments.length - 1;
-					s.distance = d;
-					s.text = this._getSegmentLabel(s, totalDistance);
-				}
 			}
+
 			for (const [i, segment] of this.segments.entries()) {
 				const unsnappedSegment = this.dragRulerUnsnappedSegments[i];
 				unsnappedSegment.startDistance = segment.startDistance;