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

Toggling curve visibility causes zoom to be reset #654

Closed
Bartimaeus- opened this issue May 28, 2022 · 2 comments
Closed

Toggling curve visibility causes zoom to be reset #654

Bartimaeus- opened this issue May 28, 2022 · 2 comments

Comments

@Bartimaeus-
Copy link
Contributor

The present behavior for toggling the visibility of waveforms in plots causes the zoom for the plot to be reset, which is tiresome when viewing a zoomed-in section on long series:

Plotjuggler.zoom.on.toggle.visibility.mp4

Notice in the video above that this behavior also breaks the functionality of keeping the zoom of all plots synchronized
image

removing the call to resetZoom() in the mouse click case for the eventFilter in PlotWidgetBase provides the desired behavior:
image

Plotjuggler.no.zoom.on.toggle.visibility.mp4

Can this behavior be changed as described or in some other way?

@facontidavide
Copy link
Owner

I am aware of this, because it is intentional.

But I might reconsider it, I guess

@Bartimaeus-
Copy link
Contributor Author

I like your solution from #655, maybe an autozoom toggle button could also be a good fit here? Or a checkbox in the preferences dialog?
image

If it is user configurable it would be nice to have it be a persistent setting so it doesn't have to be changed when you relaunch Plotjuggler

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

No branches or pull requests

2 participants