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

The order of svg is reversed for sequence diagram participant nodes #4946

Closed
ad1992 opened this issue Oct 16, 2023 · 1 comment · Fixed by #5017
Closed

The order of svg is reversed for sequence diagram participant nodes #4946

ad1992 opened this issue Oct 16, 2023 · 1 comment · Fixed by #5017
Labels
Status: Triage Needs to be verified, categorized, etc Type: Bug / Error Something isn't working or is incorrect

Comments

@ad1992
Copy link
Contributor

ad1992 commented Oct 16, 2023

Description

Hi team we are using this package to support mermaid in Excalidraw. To compute the position and dimensions I am using the SVG rendered by mermaid.
However when updating from 10.2.3 to 10.3.0 (so I can get the Actor Creation and Destruction supported 10.3.0 onwards), the order participant nodes is reversed and its only reversed for actor type Participant which makes the migration tricky.

Eg let's consider this diagram
The indexes in the diagram resembles the order of nodes on the DOM

image

Further more the order when mixing actor symbols and participants is even more confusing

image

Was there any specific reason for this change ?
Also I would like to know if the team is open to storing the dimensions and coordinates when using the API mermaid.mermaidAPI.getDiagramFromText ? That would help us in not being dependent on SVG for calculating positions / dimensions.

Steps to reproduce

Try in different versions (10.2.3 vs 10.3.0)

Screenshots

Shared above

Code Sample

No response

Setup

  • Mermaid version: 3.0.0
  • Browser and Version: [Chrome, Edge, Firefox]

Suggested Solutions

No response

Additional Context

No response

@ad1992 ad1992 added Status: Triage Needs to be verified, categorized, etc Type: Bug / Error Something isn't working or is incorrect labels Oct 16, 2023
@Shivraj-Nakum
Copy link

I would like to contribute to this issue
plz assign it to me

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Triage Needs to be verified, categorized, etc Type: Bug / Error Something isn't working or is incorrect
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants