dont render unnecessarily with opacity 0

terrastruct · May 15, 2024 · 9f5c454 · 9f5c454
1 parent 7269d30
commit 9f5c454
Show file tree

Hide file tree

Showing 6 changed files with 1,222 additions and 2 deletions.
diff --git a/d2renderers/d2svg/d2svg.go b/d2renderers/d2svg/d2svg.go
@@ -1213,7 +1213,7 @@ func drawShape(writer, appendixWriter io.Writer, diagramHash string, targetShape
 	// Closes the class=shape
 	fmt.Fprint(writer, `</g>`)
 
-	if targetShape.Icon != nil && targetShape.Type != d2target.ShapeImage {
+	if targetShape.Icon != nil && targetShape.Type != d2target.ShapeImage && targetShape.Opacity != 0 {
 		iconPosition := label.FromString(targetShape.IconPosition)
 		var box *geo.Box
 		if iconPosition.IsOutside() {
@@ -1234,7 +1234,7 @@ func drawShape(writer, appendixWriter io.Writer, diagramHash string, targetShape
 		)
 	}
 
-	if targetShape.Label != "" {
+	if targetShape.Label != "" && targetShape.Opacity != 0 {
 		labelPosition := label.FromString(targetShape.LabelPosition)
 		var box *geo.Box
 		if labelPosition.IsOutside() {

diff --git a/e2etests/testdata/txtar.txt b/e2etests/testdata/txtar.txt
@@ -213,3 +213,21 @@ ok: {
   }
   dog1 -> dog3
 }
+
+-- opacity-zero-route --
+grid: {
+  *.style.opacity: 0
+  grid-columns: 3
+  a
+  b
+  c
+  d
+  e
+  f
+  g
+  h
+
+  d.style.opacity: 1
+}
+
+out -> grid.d