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

Path tool point nudging should be in document coordinates, not viewport coordinates #2083

Closed
Keavon opened this issue Oct 30, 2024 · 1 comment · Fixed by #2095 · May be fixed by #2094
Closed

Path tool point nudging should be in document coordinates, not viewport coordinates #2083

Keavon opened this issue Oct 30, 2024 · 1 comment · Fixed by #2095 · May be fixed by #2094
Labels
Feature New feature or request Good First Issue Good for newcomers Paper Cut A small UX annoyance we should strive to improve Rust Involves Rust programming for the backend

Comments

@Keavon
Copy link
Member

Keavon commented Oct 30, 2024

If you select a point using the Path tool and nudge it to the right by 10 units with ShiftRight Arrow, starting at 100% viewport zoom but then zooming to 200%, you'll notice it takes two presses of ShiftLeft Arrow to return back to its starting place. That's because it currently is moving relative to the viewport coordinates, but it should always work relative to the document coordinates so 10 units to the right means the same thing regardless of the current level of zoom. It also shouldn't be in the layer's local coordinate space, where a shape that has a 0.1 scale factor would take 10 times as many presses to travel the intended distance. The Select tool's nudging can be referenced for this, because it works correctly in document space (not layer-local or viewport space).

@Keavon Keavon added Feature New feature or request Good First Issue Good for newcomers Rust Involves Rust programming for the backend Paper Cut A small UX annoyance we should strive to improve labels Oct 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature New feature or request Good First Issue Good for newcomers Paper Cut A small UX annoyance we should strive to improve Rust Involves Rust programming for the backend
Projects
Status: Completed This Milestone
2 participants
@Keavon and others