bpmn-io / bpmn-auto-layout

Layout BPMN diagrams, generating missing DI information.
https://bpmn-io.github.io/bpmn-auto-layout/
52 stars 41 forks source link

Allow overriding default spacing configuration #28

Closed sarialem closed 1 year ago

sarialem commented 1 year ago

Is your feature request related to a problem? Please describe.

The default separation configuration can be changed to have better arrows visibility in sequence flows, example displaying using bpmn-js:

image

After changing iLevelSeparation from 20 to 40:

image

Describe the solution you'd like

To allow overriding the default config by passing an optional config map when initializing AutoLayout

Describe alternatives you've considered

Additional context

barmac commented 1 year ago

The default spacing looks like a bug to me. I believe we should adopt the increased spacing as the built-in, and there's no need for custom configuration.

sarialem commented 1 year ago

This would work too, used your suggested approach in the PR. Probably need to label this as bug too.