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

Nudge the selected layers around the canvas with the arrow keys #201

Closed
Keavon opened this issue Jun 13, 2021 · 3 comments · Fixed by #245
Closed

Nudge the selected layers around the canvas with the arrow keys #201

Keavon opened this issue Jun 13, 2021 · 3 comments · Fixed by #245
Assignees
Labels
Feature New feature or request Good First Issue Good for newcomers Rust Involves Rust programming for the backend
Milestone

Comments

@Keavon
Copy link
Member

Keavon commented Jun 13, 2021

The arrow keys nudge (move) the selected layer(s) by one canvas pixel. When the Shift modifier key is held down, the nudges go in increments of 10 canvas pixels.

This is probably a good first issue to get acquainted with some concepts in the Rust backend. Please reach out to @TrueDoctor here or on Discord for in-depth instructions to help you get a feel for the code that this touches.

@Keavon Keavon added Feature New feature or request Good First Issue Good for newcomers Available Rust Involves Rust programming for the backend labels Jun 13, 2021
@Keavon Keavon added this to the Sprint 5 (current) milestone Jun 13, 2021
@Keavon Keavon modified the milestones: Sprint 5, Sprint 6 (current) Jul 10, 2021
@henryksloan
Copy link
Contributor

I've started working on this.

@Keavon
Copy link
Member Author

Keavon commented Jul 10, 2021

This is blocked on the inability to transform layers directly, rather than just folders (issue #243).

Some discussion from Discord:

@henryksloan: Operation::TransformLayer seems to act on Folder layers; is that the wrong way to move an individual layer?

@TrueDoctor: That is a shortcoming of the current design. We already have a fix, it just needs to get implemented.

@Keavon
Copy link
Member Author

Keavon commented Jul 11, 2021

@henryksloan you should be unblocked now courtesy of the swift work of @0HyperCube.

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 Rust Involves Rust programming for the backend
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants