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

Add pan and zoom for diagrams #334

Merged
merged 1 commit into from
Oct 21, 2023
Merged

Conversation

jp7677
Copy link
Contributor

@jp7677 jp7677 commented Oct 21, 2023

Introduce modal dialog for diagrams with pan and zoom options.
Also move export links to that dialog.

Supersedes #309

@qtzar While testing your branch I've modified quite a lot of code, so creating a new PR was kind-of easier. Could you please confirm that this PR works for you? I've also removed the config option. The dialog is a cool thing, no need to hide it behind a feature flag.

PS: I've added you as co-author to the commit since the initial work was all yours.

Introduce modal dialog for diagrams with pan and zoom options.
Also move export links to that dialog.

Co-authored-by: Declan Lynch <declan.lynch@dcsg.com>
@qtzar
Copy link
Contributor

qtzar commented Oct 21, 2023

@jp7677 Just pulled the PR locally and tested it out. It looks good to me.

Hopefully there can be a release done soon, lots of great new features waiting in the main branch...

@jp7677 jp7677 merged commit 86da272 into avisi-cloud:main Oct 21, 2023
2 checks passed
@jp7677 jp7677 deleted the svg-pan-zoom2 branch October 23, 2023 17:06
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