Merge pull request #1942 from 23danielsuh/issue1937

fix: circle shape for arrowheads no longer removes all arrows

ci/release/changelogs/next.md

@@ -8,3 +8,4 @@
 
 - Local relative icons are relative to the d2 file instead of CLI invoke path [#1924](https://github.com/terrastruct/d2/pull/1924)
 - Custom label positions weren't being read when the width was smaller than the label [#1928](https://github.com/terrastruct/d2/pull/1928)
+- Using `shape: circle` for arrowheads no longer removes all arrowheads along path in sketch mode [#1942](https://github.com/terrastruct/d2/pull/1942)

d2renderers/d2sketch/sketch.go

@@ -802,6 +802,13 @@ func ArrowheadJS(r *Runner, arrowhead d2target.Arrowhead, stroke string, strokeW
 			stroke,
 			BG_COLOR,
 		)
+	case d2target.CircleArrowhead:
+		arrowJS = fmt.Sprintf(
+			`node = rc.circle(-2, -1, 8, { strokeWidth: %d, stroke: "%s", fill: "%s", fillStyle: "solid", fillWeight: 1, seed: 5 })`,
+			strokeWidth,
+			stroke,
+			BG_COLOR,
+		)
 	}
 	return
 }

e2etests/testdata/txtar.txt

@@ -231,3 +231,30 @@ grid: {
 }
 
 out -> grid.d
+
+-- sketch-mode-circle-arrowhead --
+vars: {
+  d2-config: {
+    sketch: true
+  }
+}
+
+a
+b
+a <-> b: {
+  source-arrowhead: {
+    shape: circle
+    style.filled: false
+  }
+}
+
+x
+y
+z
+x <-> y <-> z: {
+  source-arrowhead: {
+    shape: circle
+    style.filled: false
+  }
+  direction: right
+}