-
Notifications
You must be signed in to change notification settings - Fork 195
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
Zoom plots scaling #352
base: master
Are you sure you want to change the base?
Zoom plots scaling #352
Conversation
Wonderful new feature @Indigo744 !! I have first checked the examples you provided. I think I have found a little error in the behavior in transformations_on_svg_plots.html by following this events : 1 / Zoom on the map It seems that the SVG transform is lost when applying the last zoom. Otherwise, it seems to work like a charm ! |
Damn... I though I had it working 🤦 |
I don't know why but I checked for a minimum zoom level before applying plot scaling. That should work now... 😃 |
Those SVG plots were real bitches! 🙄 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I just reviewed the code, all seems to be good to me, great great work !!!! I saw that you had to deal with SVG plots, user defined transforms, stroke width, but at the end, the result is quite perfect !
I have still found another very little bug, but I don't know if it worth fixing it : if I zoom-in and hover a plot in the same time, it seems that the stroke-width is not properly update.
Not a bug but : maybe we could keep in mind to handle the scaling for links in the same way because for now, the links will keep growing when zooming but not the plots.
There is actually a remaining bug that I should tackle... Wait for the update 😅 |
No problem, i'm waiting for it ;) |
Damn, coming back to it several month after and I can't remember what was that bug... |
Hi @Indigo744 , I will take the time to re-check your feature in all the possible ways as soon as possible. I hope we will find what was missing :) Maybe it was related to this ?
|
HI @Indigo744 , Sorry for my late answer. I ended up to take the time to re-check your great new feature, under Firefox and Chrome, and for my part, I didn't found anything else than the minor bug with the stroke width when zooming with the cursor on a plot ! |
Hello, i am still struggling to scale plots. Any help will be appriciated. |
Fix #253
Implements a feature asked by several users: automatically scale down plots on zooming.
There are 2 hard parts in this feature:
Plot scaling is disabled by default.
I've updated two examples to use plots scaling.