Graph: Set fill color for all graph text (#2707)

Alexander Eyers-Taylor · web-flow · commit a965ea4044ff · 2023-08-14T15:53:25.000Z
Grapviz no longer sets it to the default so we don't have anything to
override
diff --git a/extensions/ql-vscode/src/view/results/graph.tsx b/extensions/ql-vscode/src/view/results/graph.tsx
@@ -52,9 +52,11 @@ export function Graph({ graphData, databaseUri }: GraphProps) {
             });
           }
         }
-
         if ("fill" in d.attributes) {
-          d.attributes.fill = d.tag === "text" ? color : backgroundColor;
+          d.attributes.fill = backgroundColor;
+        }
+        if (d.tag === "text") {
+          d.attributes.fill = color;
         }
         if ("stroke" in d.attributes) {
           // There is no proper way to identify the element containing the graph (which we

Original file line number	Diff line number	Diff line change
`@@ -52,9 +52,11 @@ export function Graph({ graphData, databaseUri }: GraphProps) {`
`52`	`52`	`});`
`53`	`53`	`}`
`54`	`54`	`}`
`55`		`-`
`56`	`55`	`if ("fill" in d.attributes) {`
`57`		`- d.attributes.fill = d.tag === "text" ? color : backgroundColor;`
	`56`	`+ d.attributes.fill = backgroundColor;`
	`57`	`+ }`
	`58`	`+ if (d.tag === "text") {`
	`59`	`+ d.attributes.fill = color;`
`58`	`60`	`}`
`59`	`61`	`if ("stroke" in d.attributes) {`
`60`	`62`	`// There is no proper way to identify the element containing the graph (which we`