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

Fix Document > Clear Artboards so it doesn't also clear everything else #2177

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

moOsama76
Copy link
Contributor

@moOsama76 moOsama76 commented Jan 4, 2025

Continuation of first attempt with unmerged PR #2015

@Keavon
Copy link
Member

Keavon commented Jan 6, 2025

!build

Copy link

github-actions bot commented Jan 6, 2025

📦 Build Complete for fb422f7
https://28720137.graphite.pages.dev

@Keavon
Copy link
Member

Keavon commented Jan 6, 2025

Problem: this introduces an intermediate history step which you'll see if you open the graph and undo three times to get back when it should have taken just one step.

Other minor problem: it seems that the Artboard layer gets replaced by a Merge layer but it ends up in a different position in the graph. Ideally this wouldn't cause the layers to shift. In a case with multiple artboards each containing layers, I'm finding this even results in things overlapping in the graph after running this which isn't a suitable outcome.

Thanks, and welcome back!

@Keavon Keavon changed the title Clear Artboards Fix Document > Clear Artboards so it doesn't also clear everything else Jan 6, 2025
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