Skip to content

refactoring, add participants to sequence diagram, resolves #47 #48

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

Merged
merged 3 commits into from
Apr 9, 2025

Conversation

vareb
Copy link
Contributor

@vareb vareb commented Mar 21, 2025

Refactored into 3 files.
Improved diagram type checker that activates correct handlers.
First handler for sequence backgrounds that gets x coordinates of the click relative to the svg.

Next:
Fix so that if we change to another diagram type the handlers for the old one are deactivated

Create a model for keeping track of participants and their coordinates.
Create "addParticipant" endpoint that adds a participant at the correct position

Change-Id: Icaceadcb1468ae5ab455eaed611cf9597b6574ec
Change-Id: I35a6edb478cc634822ea840e2d996945de979707
Change-Id: I33ee850da5c91a041dff11d93b684c61588910df
@vareb vareb marked this pull request as ready for review April 7, 2025 15:31
@vareb vareb changed the title refactoring, checker for sequence diagram, first click handler refactoring, add participants to sequence diagram, resolves #47 Apr 7, 2025
@vareb vareb merged commit 0168c0c into main Apr 9, 2025
8 checks passed
@vareb vareb deleted the sequence-participant branch April 9, 2025 12:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants