Skip to content

docs: clarify indexed control compaction semantics#100

Merged
maxatwork merged 1 commit into
masterfrom
codex/clarify-indexed-control-compaction
Mar 21, 2026
Merged

docs: clarify indexed control compaction semantics#100
maxatwork merged 1 commit into
masterfrom
codex/clarify-indexed-control-compaction

Conversation

@maxatwork
Copy link
Copy Markdown
Owner

@maxatwork maxatwork commented Mar 21, 2026

Summary

  • add DOM test coverage for omitted unchecked indexed checkboxes with sparse index compaction
  • document that unchecked indexed controls do not reserve compacted array slots
  • clarify that row identity should come from another submitted field when it matters

Test Plan

  • npm test

Summary by cubic

Clarifies that omitted indexed controls do not reserve slots during array compaction, and adds a DOM test to validate this behavior. Docs also note to include another submitted field to preserve row identity when needed.

Written for commit 355f21d. Summary will update on new commits.

Copy link
Copy Markdown

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

Choose a reason for hiding this comment

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

No issues found across 3 files

@maxatwork maxatwork merged commit 2168d4b into master Mar 21, 2026
5 checks passed
@maxatwork maxatwork deleted the codex/clarify-indexed-control-compaction branch March 21, 2026 13:50
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