Skip to content

Pasting should be transfered in view if its placement wouldn't be in view #2301

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

Jump to bottom
Open
Keavon opened this issue Feb 18, 2025 · 0 comments · May be fixed by #2306
Open

Pasting should be transfered in view if its placement wouldn't be in view #2301

Keavon opened this issue Feb 18, 2025 · 0 comments · May be fixed by #2306
Labels
Good First Issue Good for newcomers Paper Cut A small UX annoyance we should strive to improve

Comments

@Keavon
Copy link
Member

Keavon commented Feb 18, 2025

Pasting a layer makes it go where it was copied from. If the viewport isn't looking at that location, and the pasted object(s) would be fully outside the view, we want to transform it/them (i.e. its/their bounding box) to the center of the viewport, by an integer number of pixels offset.

PRs with useful diffs to reference: #2212, #2177
@Keavon Keavon added Good First Issue Good for newcomers Paper Cut A small UX annoyance we should strive to improve labels Feb 18, 2025
mTvare6 added a commit to mTvare6/Graphite that referenced this issue Feb 18, 2025
@mTvare6
Transfer pasted layers to viewport center when not in viewport
73a743b 
Fixes GraphiteEditor#2301
mTvare6 added a commit to mTvare6/Graphite that referenced this issue Feb 18, 2025
@mTvare6
Transfer pasted layers to viewport center when not in viewport
bf85018 
Fixes GraphiteEditor#2301
@mTvare6 mTvare6 linked a pull request Feb 18, 2025 that will close this issue
Draft
Keavon pushed a commit to mTvare6/Graphite that referenced this issue Mar 6, 2025
@mTvare6 @Keavon
Transfer pasted layers to viewport center when not in viewport
f5adabe 
Fixes GraphiteEditor#2301
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Good First Issue Good for newcomers Paper Cut A small UX annoyance we should strive to improve
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Transfer pasted layers to viewport center when not in viewport mTvare6/Graphite
1 participant
@Keavon