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

Removing label from sequence diagram connection causes browser crash #47

Open
melink14 opened this issue Oct 21, 2022 · 2 comments
Open
Assignees
Labels
bug Something isn't working

Comments

@melink14
Copy link

I started with the basic example and deleted the ':Hello Bob!' part of the first line.

After that, the graph stops rendering due to 'memory access out of bounds'

Refreshing fixes it.

@ArthurSonzogni ArthurSonzogni self-assigned this Oct 21, 2022
@ArthurSonzogni ArthurSonzogni added the bug Something isn't working label Oct 21, 2022
@ArthurSonzogni
Copy link
Owner

Thank you!
I will take a look, except if you want to find out yourself what's causing this?

@melink14
Copy link
Author

Thanks for looking.

On looking deeper, the stack trace I was seeing wasn't due to the freezing problem but instead appearing on inital load so it's probably a red herring.

Also, testing now on my home computer, I see that the rendering does freeze once you delete the : but starts working again once it's added back.

If the cause is just that you must have a label for connections then maybe this is just a small usability problem. I was confused when it stopped updating but it's minor!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants