Skip to content

Culling viewport#3

Closed
kaznaan wants to merge 1 commit intokaz/add-grid-4.3from
kaz/cull-viewport
Closed

Culling viewport#3
kaznaan wants to merge 1 commit intokaz/add-grid-4.3from
kaz/cull-viewport

Conversation

@kaznaan
Copy link
Copy Markdown
Collaborator

@kaznaan kaznaan commented Feb 16, 2026

Culling viewport in tldraw side to reduce amount of DOM.

@kaznaan kaznaan requested a review from theop-luma February 16, 2026 15:34
@cursor
Copy link
Copy Markdown

cursor Bot commented Feb 16, 2026

PR Summary

Medium Risk
Changes which shapes are mounted/rendered based on viewport culling, which may cause offscreen shapes/indicators to disappear unexpectedly or introduce edge cases around selection/editing visibility and re-render behavior.

Overview
Adds a new public Editor.getVisibleRenderingShapes() computed that filters getRenderingShapes() by the current culling set (getCulledShapes()).

Updates DefaultCanvas and DefaultShapeIndicators to use this new method instead of getRenderingShapes(), reducing mounted DOM/indicators to viewport-visible shapes, and updates the API report accordingly.

Written by Cursor Bugbot for commit 19f8c52. This will update automatically on new commits. Configure here.

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