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

Automatically place layers into the artboard they're drawn inside of #2110

Merged
merged 5 commits into from
Dec 21, 2024

Conversation

james-ryans
Copy link
Contributor

Part of #1507

@james-ryans james-ryans marked this pull request as ready for review November 25, 2024 16:24
@Keavon Keavon force-pushed the new-layer-artboard-bounds branch from 377b910 to 3bcefaf Compare December 20, 2024 22:03
@Keavon
Copy link
Member

Keavon commented Dec 21, 2024

!build

Copy link

📦 Build Complete for 3bcefaf
https://b1dcab56.graphite.pages.dev

@Keavon Keavon changed the title Place layers within the bounds of any artboard Automatically place layers into the artboard they're drawn inside of Dec 21, 2024
Copy link
Member

@Keavon Keavon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@Keavon Keavon merged commit 3be7505 into GraphiteEditor:master Dec 21, 2024
3 of 4 checks passed
@Keavon
Copy link
Member

Keavon commented Jan 1, 2025

Hi @james-ryans, I've just discovered a regression introduced by this PR. Would you be able to work on solving it in a new PR, please? If you're not able to do so in the next few days, please let me know and I'll try to find someone else to work on it, but you're probably the most familiar with the code so it'd be great if you do have the time. Thank you!

Problem description: when a nested layer is selected, creating new shapes with the drawing tools should put the new layers alongside (as siblings of) the selected layer. The regression results in those always being drawn at the top level, just inside the artboard (if there is one, otherwise at the document root).

@Keavon
Copy link
Member

Keavon commented Jan 9, 2025

@james-ryans Hello, just checking up to see your plans. I saw you reacted to the previous message. Since this introduces a somewhat serious quality-of-life issue for drawing inside of folders, I may revert this PR within the next few days so the original feature can be re-attempted now that we know this is a regression to watch out for (which I failed to find during my initial QA tests). But I'm just commenting to see if, perhaps, you've been already working on a solution and are nearly ready to open a PR for it. In that case, I'll hold off for a few days before reverting. What's best in regards to your plans? Thank you!

Keavon added a commit that referenced this pull request Jan 10, 2025
@Keavon
Copy link
Member

Keavon commented Jan 10, 2025

Turns out it wasn't overly difficult to fix, so I implemented it myself (in the commit listed in the history directly above this comment) out of concern for timing. Now I can do another production deploy so users can access an editor without this breakage again. When you have more time on your hands to keep contributing, there is plenty available, just ping me and I'm happy to assign more feature requests for you to take a look at. Thanks again for this valuable feature!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants