From 405fdbe57f65ca419167226576f73139053dc355 Mon Sep 17 00:00:00 2001 From: Alexander Wang Date: Sat, 31 Dec 2022 12:17:05 -0800 Subject: [PATCH 1/2] don't add appendix seperator without appendix --- d2renderers/d2svg/appendix/appendix.go | 3 +++ 1 file changed, 3 insertions(+) diff --git a/d2renderers/d2svg/appendix/appendix.go b/d2renderers/d2svg/appendix/appendix.go index 47e7a7ba12..7237075ab2 100644 --- 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(`%s From 2b2c9120802b358eb637cd0b27f47a51519ea272 Mon Sep 17 00:00:00 2001 From: Alexander Wang Date: Sat, 31 Dec 2022 12:17:52 -0800 Subject: [PATCH 2/2] changelog --- ci/release/changelogs/next.md | 2 ++ 1 file changed, 2 insertions(+) diff --git a/ci/release/changelogs/next.md b/ci/release/changelogs/next.md index f3c0d2a774..dc555bd027 100644 --- 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)