Skip to content

fix: bound spreadsheet width and skip empty spill scans - #1095

Merged
andiwand merged 1 commit into
mainfrom
review/77-sheet-render-bounds
Oct 5, 2026
Merged

andiwand merged 1 commit into
mainfrom
review/77-sheet-render-bounds

Conversation

@andiwand

@andiwand andiwand commented Oct 5, 2026 •

Copy link
Copy Markdown
Member

🤖 Generated with Claude Code

A wide sheet can no longer exceed the cell budget by forcing one oversized row: cap columns first, then rows, retaining the existing minimum of one cell. Blank cells also stop scanning the remaining blank span for overflow, removing quadratic work on wide sparse sheets and unnecessary clipping rules.

Validation: 128 HTML, URI and style checks pass. Extends the existing budget and spill tests without adding another test case.

@andiwand
andiwand force-pushed the review/76-editor-unicode-deletion branch from e832c5a to 2401787 Compare October 5, 2026 20:04
Base automatically changed from review/76-editor-unicode-deletion to main October 5, 2026 20:08
@andiwand
andiwand force-pushed the review/77-sheet-render-bounds branch from 910b23f to 95acd1e Compare October 5, 2026 20:13
@andiwand
andiwand merged commit 2ac4253 into main Oct 5, 2026
23 checks passed
@andiwand
andiwand deleted the review/77-sheet-render-bounds branch October 5, 2026 20:20
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