search

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

72 happy path first #73

Closed abdul99ahad closed 1 month ago

abdul99ahad commented 1 month ago

Closes #72

Proposed Changes

The primary path will always come first, with secondary connections following below. image ### Checklist To ensure you provided everything we need to look at your PR: * [x] **Brief textual description** of the changes present * [x] **Visual demo** attached * [ ] **Steps to try out** present, i.e. [using the `@bpmn-io/sr` tool](https://github.com/bpmn-io/sr) * [x] Related issue linked via `Closes {LINK_TO_ISSUE}` or `Related to {LINK_TO_ISSUE}`
abdul99ahad commented 1 month ago

@nikku Confused about the strategy of happy path first after Till's comment https://github.com/bpmn-io/bpmn-auto-layout/issues/50#issuecomment-2377248858. Do we consider connections from top or not?

nikku commented 1 month ago

Do we consider connections from top or not?

I consider that we fully understand our strategy, and take it from there. For me "back loop on top" is nothing that I've seen applied in practice, and it makes it harder to understand the happy path.