constellation icon indicating copy to clipboard operation
constellation copied to clipboard

Only reset view/camera position on related graph.

Open OrionsGuardian opened this issue 1 year ago • 1 comments

Prerequisites

  • [ ] Reviewed the checklist

  • [ ] Reviewed feedback from the "Sonar Cloud" bot. Note that you have to wait for the "CI / Unit Tests") to complete first. Failed Unit tests can be debugged by adding the label "verbose logging" to the GitHub PR.

Description of the Change

Modified the plugins which alter the camera position (zoom level) to check if the related graph is active before doing an animated camera zoom. If the related graph is not the active one, the new camera position will be set directly in the related graph, skipping the animation process.

Alternate Designs

Why Should This Be In Core?

Benefits

If you are working on multiple graphs and run a plugin on one of those graphs, and then switch to a different graph while waiting for the plugin to complete. The current graph being viewed will not suddenly have it's zoom level changed when the first graph's plugin has completed.

Possible Drawbacks

Verification Process

Applicable Issues

OrionsGuardian avatar Sep 19 '22 06:09 OrionsGuardian