-
-
Notifications
You must be signed in to change notification settings - Fork 267
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
[Feature Request] Zoom and panning in html report #843
Comments
Are you offering to provide the JS code? If so, a draft PR would work. |
Yes I already have js code that implements it, and now I study your code to find the right place for it, ad i suppose it should be in |
Yes, that's correct. Thanks for taking a look at it. |
So now I can prepare pull request? |
Yes please, a draft PR in a branch of forked Archi code is the way to go. Thanks. |
Sorry for long time for implementing, but finally I've got pull request about this feature #882 |
Thanks for this, I merged it. |
@Ujifman Hi, I've found a problem with const mouseDownHandler = function (e) {
e.preventDefault(); <-------------------- Here! Would you be able to take a look at this? BTW - I've edited the file to support IE11 |
One thing I tried that seems to work is removing the
|
I've created a branch |
@Phillipus Hi, sorry, I've missed alert from GitHub, had some crazy days at work. As I see you already fixed the issue, do you need some help? |
@Ujifman Hi, I just wanted to check it with you, first. Do you see any problems with my proposed fix? |
Looks nice! I don't see any problems. |
Thanks for the feedback. I'll commit it, |
Hi, this is causing problems as noted by @evlibra: 5f81eed#commitcomment-95334110 b8c4163#commitcomment-95333906 We should reconsider this one for Archi 5 and find a way to only pan when the mouse is down on a diagram. If not we should remove this feature for now and come back to it. @Ujifman Can you find a way to do this? |
* add Ctrl + wheel zoom in for diagram only instead of whole page * add panning by mouse left click over the diagram
I think I've fixed this issue now. @evlibra: please test. |
Thanks @Phillipus. Yes, the issue is fixed now - frames beyond the diagram are not impacted. The only question is whether grabbing shall be allowed only within boundaries of diagram elements (see area in green on the screenshot), or within whole diagram frame, meaning also empty area at the right from the diagram elements. At the moment is is only within diagram's elements (V), but intuitively one might expect also grab at right from it (?). See the screenshot |
I'm sorry but I can't spend any more time on this issue, I've tried my best to make it work based on @Ujifman 's PR. If it continues to be a problem I will remove it. |
This is now removed. I've left the mouse scroll wheel feature in. Please resubmit a working PR that we might consider for Archi 5.1. |
@Phillipus I'll try to find out a way to fix it. |
Hello, I found two features that will increase usability of html report, when you in
views diagrams
.My idea is just to add some js code to report building tool, that will implement described behavior.
The text was updated successfully, but these errors were encountered: