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

CucaDiagramFileMakerSmetana margin strangeness #497

Closed
matthew16550 opened this issue Mar 20, 2021 · 1 comment · Fixed by #499
Closed

CucaDiagramFileMakerSmetana margin strangeness #497

matthew16550 opened this issue Mar 20, 2021 · 1 comment · Fixed by #499

Comments

@matthew16550
Copy link
Collaborator

The default margins when using CucaDiagramFileMakerSmetana have a similar problem to #494 but it's not a recent regression, both v1.2021.2 and latest are identical for this.

comparison-latest

And when we specify a margin there is extra space added at the top. e.g. these rows use margin 0, margin 20 & margin 40:

margin-comparison-latest

Think the top space issue can be fixed by using LimitFinder directly inCucaDiagramFileMakerSmetana.createFileLocked() instead of via ImageBuilder.getFinalDimension().

I'm cautious about backwards compatibility here, can the current behaviour be changed?

@arnaudroques
Copy link
Contributor

I'm cautious about backwards compatibility here, can the current behaviour be changed?

Yes, the current behaviour for Smetana could (and should) be changed.
Smetana is really a beta feature: it uses a Java implementation of dot.
(That is, you do not need to install GraphViz to run PlantUML!)

So margin was clearly not managed by Smetana.
Now that the code is clearer, it's time to fix that !

Thanks!

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 a pull request may close this issue.

2 participants