Skip to content

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

Merged
Keavon merged 5 commits intoGraphiteEditor:masterfrom
james-ryans:new-layer-artboard-bounds
Dec 21, 2024
Merged

Automatically place layers into the artboard they're drawn inside of#2110
Keavon merged 5 commits intoGraphiteEditor:masterfrom
james-ryans:new-layer-artboard-bounds

Conversation

@james-ryans
Copy link
Copy Markdown
Contributor

@james-ryans james-ryans commented Nov 16, 2024

Partly closes #1507

@james-ryans james-ryans marked this pull request as ready for review November 25, 2024 16:24
Signed-off-by: James Ryans <james.ryans2012@gmail.com>
Signed-off-by: James Ryans <james.ryans2012@gmail.com>
Signed-off-by: James Ryans <james.ryans2012@gmail.com>
Signed-off-by: James Ryans <james.ryans2012@gmail.com>
Signed-off-by: James Ryans <james.ryans2012@gmail.com>
@Keavon Keavon force-pushed the new-layer-artboard-bounds branch from 377b910 to 3bcefaf Compare December 20, 2024 22:03
@Keavon
Copy link
Copy Markdown
Member

Keavon commented Dec 21, 2024

!build

@github-actions
Copy link
Copy Markdown

📦 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
Copy Markdown
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
@Keavon
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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.

Smarter placement of layers into the artboards that bound them

2 participants