mirrors/Graphite
1
0
Fork 0
mirror of https://github.com/GraphiteEditor/Graphite.git synced 2025-07-08 00:05:00 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

Fix Select tool resizing with Shift held allowing the constrained aspect ratio to change when snapping (#2441)

Browse source 
Fix bug in constrained snap
This commit is contained in:
James Lindsay 2025-03-17 10:18:05 +00:00 • committed by GitHub
parent bd97c15da8
commit cabc027cda
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 16 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

17
editor/src/messages/tool/common_functionality/transformation_cage.rs
View file

@ -206,7 +206,22 @@ impl SelectedEdges {
}
let snapped_bounds = bounds_to_doc.inverse().transform_point2(snapped.snapped_point_document);
let mut scale_factor = (snapped_bounds - pivot) / (updated - pivot);
let new_from_pivot = snapped_bounds - pivot; // The new vector from the snapped point to the pivot
let original_from_pivot = updated - pivot; // The original vector from the point to the pivot
let mut scale_factor = new_from_pivot / original_from_pivot;
// Constrain should always scale by the same factor in x and y
if constrain {
// When the point is on the pivot, we simply copy the other axis.
if original_from_pivot.x.abs() < 1e-5 {
scale_factor.x = scale_factor.y;
} else if original_from_pivot.y.abs() < 1e-5 {
scale_factor.y = scale_factor.x;
}
debug_assert!((scale_factor.x - scale_factor.y).abs() < 1e-5);
}
if !(self.left || self.right || constrain) {
scale_factor.x = 1.
}