Skip to content

Add virtual scrolling to the Layers panel, sending details and thumbnails only for rows in view - #4768

Merged
Keavon merged 1 commit into
masterfrom
layers-panel-virtual-scrolling
Oct 21, 2026
Merged

Keavon merged 1 commit into
masterfrom
layers-panel-virtual-scrolling

Conversation

@Keavon

@Keavon Keavon commented Oct 21, 2026 •

Copy link
Copy Markdown
Member

This also fixes the memory leak where old thumbnails were never evicted.

@cubic-dev-ai cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

All reported issues were addressed across 11 files

Reply with feedback, questions, or to request a fix.

Re-trigger cubic

Comment thread frontend/src/stores/portfolio.ts Outdated
Comment thread editor/src/messages/portfolio/document/utility_types/nodes.rs Outdated
Comment thread editor/src/node_graph_executor.rs Outdated
Comment thread frontend/src/components/panels/Layers.svelte Outdated
Comment thread editor/src/node_graph_executor.rs Outdated
Comment thread frontend/src/components/panels/Layers.svelte Outdated

@cubic-dev-ai cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

All reported issues were addressed across 11 files (changes from recent commits).

Reply with feedback, questions, or to request a fix.

Re-trigger cubic

Comment thread editor/src/node_graph_executor.rs Outdated
@Keavon
Keavon force-pushed the layers-panel-virtual-scrolling branch from 7b0ef97 to f54d847 Compare October 21, 2026 04:12
@Keavon
Keavon merged commit c6ac909 into master Oct 21, 2026
10 checks passed
@Keavon
Keavon deleted the layers-panel-virtual-scrolling branch October 21, 2026 04:25

This branch was successfully deployed

1 active deployment
graphite-dev (Preview) — f54d8474 Deployed Oct 21, 2026 by github-actions[bot]
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