Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cli: Improve multiboard output #923

Merged
merged 5 commits into from
Feb 27, 2023

Conversation

nhooyr
Copy link
Contributor

@nhooyr nhooyr commented Feb 27, 2023

  • Boards with subboards are now selfcontained folders with index.svg/png as
    the root board render.
  • Boards that are only containers of other boards are not rendered. For example a
    scenario with no modifications and only steps only has its steps
    rendered.
  • Boards with sibling boards of another type are rendered under a
    subdirectory indicating their type to separate them. For example a
    board with layers and scenarios has its layers rendered into subfolder
    layers and scenarios into subfolder scenarios.

cc @berniexie see BoardContainer field on d2target.Board for the field
you were looking for to skip renders for PDFs too.

@nhooyr nhooyr requested review from alixander, berniexie and a team February 27, 2023 21:29
@nhooyr nhooyr force-pushed the layers-containers-97eb branch 3 times, most recently from e36efc9 to c86f0c0 Compare February 27, 2023 21:38
- Boards with subboards are now selfcontained folders with index.svg/png as
  the root board render.
- Boards that are only containers of other boards are not rendered. For example a
  scenario with no modifications and only steps only has its steps
  rendered.
- Boards with sibling boards of another type are rendered under a
  subdirectory indicating their type to separate them. For example a
  board with layers and scenarios has its layers rendered into subfolder
  layers and scenarios into subfolder scenarios.

cc @berniexie see BoardContainer field on d2target.Board for the field
you were looking for to skip renders for PDFs too.
.gitignore Show resolved Hide resolved
d2renderers/d2sketch/testdata/animated_dark/sketch.exp.svg Outdated Show resolved Hide resolved
d2compiler/compile.go Outdated Show resolved Hide resolved
@nhooyr nhooyr merged commit 233a3fd into terrastruct:master Feb 27, 2023
@nhooyr nhooyr deleted the layers-containers-97eb branch February 27, 2023 22:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants