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

[Plots] Reload button doesn't respect current zoom level #7929

Open
3 of 7 tasks
jvigliotta opened this issue Nov 13, 2024 · 1 comment
Open
3 of 7 tasks

[Plots] Reload button doesn't respect current zoom level #7929

jvigliotta opened this issue Nov 13, 2024 · 1 comment

Comments

@jvigliotta
Copy link
Contributor

jvigliotta commented Nov 13, 2024

related to #7928

Summary

When zoomed into a plot, clicking the Reload action will reset the zoom level.

Expected vs Current Behavior

Would like the zoom level to be respected and the data in the zoomed in portion of the plot to be displayed correctly (not at the same "resolution" as the time conductor range).

A good point about this: #7929 (comment)

Steps to Reproduce

  1. Zoom in to a plot
  2. Select the Reload option from the 3-dot menu
  3. Notice the plot is reloaded, but the zoom level is thrown away

Environment

  • Open MCT Version:
  • Deployment Type:
  • OS:
  • Browser:

Impact Check List

  • Data loss or misrepresented data?
  • Regression? Did this used to work or has it always been broken?
  • Is there a workaround available? zoom back in, but not desirable
  • Does this impact a critical component?
  • Is this just a visual bug with no functional impact?
  • Does this block the execution of e2e tests?
  • Does this have an impact on Performance?

Additional Information

Reload action in the 3-dot menu
Screenshot 2024-11-13 at 11 13 10 AM

@mudinthewater
Copy link

Thanks! Worth mentioning the rationale in having this respect temporary zoom level is that you can achieve the same workflow by using the reset zoom level button on the top left. Resetting zoom level will also re-query, although reload appears to clear the plot object at a higher level, ordinary users will not know or care about this and will just see that both reset zoom and reload do the same thing if you're zoomed in.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants