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.

Sign up for GitHub

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

Jump to bottom

[FEAT] Review Black Box Pool Rendering #718

Closed
tbouffard opened this issue Oct 9, 2020 · 1 comment
Closed

[FEAT] Review Black Box Pool Rendering #718

tbouffard opened this issue Oct 9, 2020 · 1 comment
Labels
BPMN rendering Something about the way the lib is rendering BPMN elements enhancement New feature or request good first issue Good for newcomers hacktoberfest Can be done during the Hacktoberfest
Milestone
BPMN rendering im...

Comments

@tbouffard
Copy link
Member

tbouffard commented Oct 9, 2020 •
edited
Loading

Is your feature request related to a problem? Please describe.
Decide if we still want to keep the horizontal and vertical display with the label separator zone.
We may always display the label horizontally without any label line separator. This could also be a rendering option (see #2659).

Describe the solution you'd like
The initial implementation #551 displays a label line separator as described in #516 (comment).
The BPMN Spec let vendor decide how they want to render Black Box Pool

If the Pool is a black box (i.e., does not contain a Process), then the label for the Pool MAY be placed
anywhere within the Pool without a single line separator.

BPMN Spec 2.0.2 page 111 (pdf page 141)

⚠️ Thanks to discuss with us on this issue to propose your different renderings before to implement one of them.
@tbouffard tbouffard added enhancement New feature or request BPMN rendering Something about the way the lib is rendering BPMN elements labels Oct 9, 2020
@tbouffard tbouffard added this to the BPMN rendering improvements milestone Oct 9, 2020
@csouchet csouchet added hacktoberfest Can be done during the Hacktoberfest good first issue Good for newcomers labels Sep 22, 2021
@tbouffard tbouffard mentioned this issue May 2, 2023
Closed
@tbouffard
Copy link
Member Author

tbouffard commented Jul 11, 2023 •
edited
Loading

Decision after discussions with @csouchet
There is no traction for this feature, so we are not going to implement it.
Feel free to open a new issue to explain why you may need it

@tbouffard tbouffard closed this as not planned Won't fix, can't repro, duplicate, stale Jul 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
BPMN rendering Something about the way the lib is rendering BPMN elements enhancement New feature or request good first issue Good for newcomers hacktoberfest Can be done during the Hacktoberfest
Projects
None yet
Milestone
BPMN rendering improvements
Development

No branches or pull requests
2 participants
@csouchet @tbouffard