From f166077e7b5f38313222d0b51f05177b2edc8bf7 Mon Sep 17 00:00:00 2001
From: Gavin Nishizawa <gavin@terrastruct.com>
Date: Fri, 16 Dec 2022 17:15:37 -0800
Subject: [PATCH] always define the referenced mask

---
 d2renderers/d2svg/d2svg.go | 25 ++++++++++++-------------
 1 file changed, 12 insertions(+), 13 deletions(-)

diff --git a/d2renderers/d2svg/d2svg.go b/d2renderers/d2svg/d2svg.go
index 6e31221c8..cf844c234 100644
--- a/d2renderers/d2svg/d2svg.go
+++ b/d2renderers/d2svg/d2svg.go
@@ -1024,19 +1024,18 @@ func Render(diagram *d2target.Diagram, pad int) ([]byte, error) {
 		}
 	}
 
-	if len(labelMasks) > 0 {
-		fmt.Fprint(buf, strings.Join([]string{
-			fmt.Sprintf(`<mask id="%s" maskUnits="userSpaceOnUse" x="0" y="0" width="%d" height="%d">`,
-				labelMaskID, w, h,
-			),
-			fmt.Sprintf(`<rect x="0" y="0" width="%d" height="%d" fill="white"></rect>`,
-				w,
-				h,
-			),
-			strings.Join(labelMasks, "\n"),
-			`</mask>`,
-		}, "\n"))
-	}
+	// Note: we always want this since we reference it on connections even if there end up being no masked labels
+	fmt.Fprint(buf, strings.Join([]string{
+		fmt.Sprintf(`<mask id="%s" maskUnits="userSpaceOnUse" x="0" y="0" width="%d" height="%d">`,
+			labelMaskID, w, h,
+		),
+		fmt.Sprintf(`<rect x="0" y="0" width="%d" height="%d" fill="white"></rect>`,
+			w,
+			h,
+		),
+		strings.Join(labelMasks, "\n"),
+		`</mask>`,
+	}, "\n"))
 
 	embedFonts(buf)