Merge pull request #1848 from alixander/d2oracle-arrowhead-fill-2

followup fix to #1847

Author: alixander

d2oracle/edit.go

@@ -647,8 +647,14 @@ func _set(g *d2graph.Graph, baseAST *d2ast.Map, key string, tag, value *string)
 			case "source-arrowhead", "target-arrowhead":
 				var arrowhead *d2graph.Attributes
 				if reservedKey == "source-arrowhead" {
+					if edge.SrcArrowhead != nil {
+						attrs = *edge.SrcArrowhead
+					}
 					arrowhead = edge.SrcArrowhead
 				} else {
+					if edge.DstArrowhead != nil {
+						attrs = *edge.DstArrowhead
+					}
 					arrowhead = edge.DstArrowhead
 				}
 				if arrowhead != nil {
@@ -669,6 +675,12 @@ func _set(g *d2graph.Graph, baseAST *d2ast.Map, key string, tag, value *string)
 							arrowhead.Label.MapKey.SetScalar(mk.Value.ScalarBox())
 							return nil
 						}
+					case "style":
+						reservedTargetKey = mk.Key.Path[len(mk.Key.Path)-1].Unbox().ScalarString()
+						if inlined(attrs.Style.Filled) {
+							attrs.Style.Filled.MapKey.SetScalar(mk.Value.ScalarBox())
+							return nil
+						}
 					}
 				}
 			case "style":

d2oracle/edit_test.go

@@ -1562,6 +1562,42 @@ a.b -> a.c: {style.animated: true}
   target-arrowhead.shape: diamond
   target-arrowhead.style.filled: true
 }
+`,
+		},
+		{
+			name: "edge-arrowhead-filled/4",
+			text: `x -> y: {
+	target-arrowhead.shape: diamond
+  target-arrowhead.style.filled: true
+}
+`,
+			key:   `(x -> y)[0].target-arrowhead.style.filled`,
+			value: go2.Pointer(`false`),
+
+			exp: `x -> y: {
+  target-arrowhead.shape: diamond
+  target-arrowhead.style.filled: false
+}
+`,
+		},
+		{
+			name: "edge-arrowhead-filled/5",
+			text: `x -> y: {
+	target-arrowhead.shape: diamond
+	target-arrowhead.style: {
+    filled: false
+  }
+}
+`,
+			key:   `(x -> y)[0].target-arrowhead.style.filled`,
+			value: go2.Pointer(`true`),
+
+			exp: `x -> y: {
+  target-arrowhead.shape: diamond
+  target-arrowhead.style: {
+    filled: true
+  }
+}
 `,
 		},
 		{