Skip to content

fix: framerate-independent global scale speed #33

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.

Sign up for GitHub

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

Merged
manaporkun merged 1 commit into from
Jun 1, 2026
Merged

fix: framerate-independent global scale speed #33

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 7 additions & 2 deletions ...rkunmanap.runtime-transform-handles/Runtime/Scripts/HandleComponents/Scale/ScaleGlobal.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,9 @@ namespace TransformHandles
/// </summary>
public class ScaleGlobal : HandleBase
{
private const float MouseSensitivity = 2f;
// Scale delta per pixel dragged. Chosen to match the previous feel at 60 fps
// (the old code used 2f * Time.deltaTime, i.e. ~2/60 per pixel at 60 fps).
private const float MouseSensitivity = 0.0333f;

[SerializeField] private Color defaultColor;
[SerializeField] private MeshRenderer cubeMeshRenderer;
Expand Down Expand Up @@ -37,7 +39,10 @@ public void Initialize(Handle handle, Vector3 axis)
public override void Interact(Vector3 previousPosition)
{
var mouseVector = (Vector3)InputWrapper.MousePosition - previousPosition;
var d = (mouseVector.x + mouseVector.y) * Time.deltaTime * MouseSensitivity;
// mouseVector is the per-frame pixel delta; the total scale change should depend on how
// far the mouse moved, not on the frame rate. Multiplying by Time.deltaTime made scaling
// speed framerate-dependent (higher fps -> smaller dt -> slower scaling). Drop it.
var d = (mouseVector.x + mouseVector.y) * MouseSensitivity;
delta += d;
ParentHandle.target.localScale = _startScale + Vector3.Scale(_startScale, _axis) * delta;

Expand Down
Loading