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

Missing default margin/padding on direction: right/left with 3 embedded shapes? #968

Closed
czardien opened this issue Mar 3, 2023 · 3 comments
Assignees

Comments

@czardien
Copy link

czardien commented Mar 3, 2023

Hi,

I'm seeing a slightly different behaviour in the margin/padding between directions up/down and left/right, where a margin/padding will exist in the former but not in the latter, when have more than 2 embedded shapes.

To see the behaviour, one can compare the output of the following script:

direction: up

A
B
B.C

A.a
B.C.b

A.a -> B.C.b

... and:

direction: right

A
B
B.C

A.a
B.C.b

A.a -> B.C.b

... see attached for the results, which are interchangeable for {up|down} and {left|right}.

We can clearly see an "expected" margin/padding in the former script, which we can't see in the latter.

right-no-margin-padding
up

@cyborg-ts cyborg-ts added this to D2 Mar 3, 2023
@czardien czardien changed the title Missing default margin/padding on direction: right/left? Missing default margin/padding on direction: right/left with 3 embedded shapes? Mar 3, 2023
@alixander
Copy link
Collaborator

yes i think related to #925

we'll get this fixed this release (probably release tonight). thank you for reporting.

@czardien
Copy link
Author

czardien commented Mar 3, 2023

I should mention I'm running d2 version 0.2.2 which I installed from the Arch package.

$ d2 -v
0.2.2

@gavin-ts
Copy link
Contributor

@github-project-automation github-project-automation bot moved this to Done in D2 Mar 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

No branches or pull requests

3 participants