Merge pull request #582 from alixander/dont-add-appendix-seperator

don't add appendix seperator without appendix
terrastruct · Dec 31, 2022 · dbc09d1 · dbc09d1
2 parents ae94c14 + 2b2c912
commit dbc09d1
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/ci/release/changelogs/next.md b/ci/release/changelogs/next.md
@@ -3,3 +3,5 @@
 #### Improvements 🧹
 
 #### Bugfixes ⛑️
+
+- Appendix seperator line no longer added to PNG export when appendix doesn't exist. [#582](https://github.com/terrastruct/d2/pull/582)
diff --git a/d2renderers/d2svg/appendix/appendix.go b/d2renderers/d2svg/appendix/appendix.go
@@ -153,6 +153,9 @@ func generateAppendix(diagram *d2target.Diagram, ruler *textmeasure.Ruler, svg s
 			}
 		}
 	}
+	if len(lines) == 0 {
+		return "", 0, 0
+	}
 	totalHeight += SPACER
 
 	return fmt.Sprintf(`<g x="%d" y="%d" width="%d" height="100%%">%s</g>
Original file line number	Diff line number	Diff line change
Expand Up		@@ -3,3 +3,5 @@
		#### Improvements 🧹

		#### Bugfixes ⛑️

		- Appendix seperator line no longer added to PNG export when appendix doesn't exist. [#582](https://github.com/terrastruct/d2/pull/582)