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

Instance tables refactor part 4: replace ArtboardGroups with multi-row Instances<Artboard> #2265

Merged
merged 5 commits into from
Mar 3, 2025

Conversation

Keavon
Copy link
Member

@Keavon Keavon commented Feb 4, 2025

Part 4 of #1834.

Flattens GraphicGroup and ArtboardGroup structs, placing their Vec data into rows of their Instances<GraphicElement> and Instances<Artboard> tables.

This PR is based on top of #2256. The first commit in the chain of this PR's changes is 39f98b1. This PR is blocked on that PR being merged first.

@Keavon Keavon marked this pull request as ready for review March 3, 2025 01:22
@Keavon Keavon changed the title Instance tables refactor part 4: flatten groups to use the instance tables instead Instance tables refactor part 4: flatten ArtboardGroups in lieu of Instances<Artboard> with multiple rows Mar 3, 2025
@Keavon Keavon changed the title Instance tables refactor part 4: flatten ArtboardGroups in lieu of Instances<Artboard> with multiple rows Instance tables refactor part 4: flatten ArtboardGroups in lieu of multi-row Instances<Artboard> Mar 3, 2025
@Keavon Keavon changed the title Instance tables refactor part 4: flatten ArtboardGroups in lieu of multi-row Instances<Artboard> Instance tables refactor part 4: replace ArtboardGroups with multi-row Instances<Artboard> Mar 3, 2025
@Keavon Keavon enabled auto-merge (squash) March 3, 2025 01:30
@Keavon Keavon disabled auto-merge March 3, 2025 01:30
@Keavon Keavon merged commit 0158be3 into master Mar 3, 2025
5 checks passed
@Keavon Keavon deleted the instance-tables-pt4 branch March 3, 2025 01:30
Keavon added a commit that referenced this pull request Mar 6, 2025
…w Instances<Artboard> (#2265)

* Clean up dyn_any usages

* Migrate ArtboardGroup to ArtboardGroupTable (not yet flattened)

* Reorder graphical data imports

* Flatten and remove ArtboardGroup in favor of ArtboardGroupTable

* Fix test
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.

1 participant