Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Purpose
Small PR related to DYN-7466 and feedback from the last Archilizer Check-in regarding nodes/groups appearing too large when zoom-to-fit is applied.
I've added a
paddingFactor
of 1.5 toWorkspaceViewModel.FitViewInternal
. This adjustment affects zoom-to-fit behavior for node or group selections in TuneUp and Graph Node Manager, as well as for the Zoom to Fit button in workspace.I think keeping
paddingFactor
as a local variable is appropriate, and the value of 1.5 seems reasonable. However, please let me know if this should be changed.Before ---> ---> ---> After
Declarations
Check these if you believe they are true
*.resx
filesRelease Notes
Padding factor added to zoom-to-fit, affecting TuneUp, Graph Node Manager, and the Zoom to Fit button.
Reviewers
@reddyashish
@QilongTang
FYIs
@dnenov
@Amoursol