Skip to content

test(behavior): SDT inside-edge backspace/delete parity (SD-3237/SD-3218)#3556

Merged
caio-pizzol merged 2 commits into
mainfrom
caio/sdt-inline-inside-edge-parity
May 28, 2026
Merged

test(behavior): SDT inside-edge backspace/delete parity (SD-3237/SD-3218)#3556
caio-pizzol merged 2 commits into
mainfrom
caio/sdt-inline-inside-edge-parity

Conversation

@caio-pizzol
Copy link
Copy Markdown
Contributor

Closes the inside-edge half of the inline boundary keyboard matrix. The outside-edge half (select-then-delete) landed in #3552/#3554; this covers editing from inside the control edge, which Word treats very differently.

Consumes two word-api parity contracts: sdt/inline-backspace-inside-trailing and sdt/inline-delete-inside-leading.

  • Editable modes (unlocked, sdtLocked): each press deletes one character from the respective edge; wrapper preserved.
  • Content-locked modes (contentLocked, sdtContentLocked): blocked, content unchanged.

All four lock modes exercised per scenario. Unlike the outside-edge tests there is no divergence here - SuperDoc matches Word, so no test.fail cases. Branches off main, not stacked. No raw Word data committed.

Verified: 24 passed across chromium, firefox, webkit.

…SD-3218)

Closes the inside-edge half of the boundary keyboard matrix (the outside-edge
half landed in #3552/#3554). Consumes the word-api contracts
inline-backspace-inside-trailing and inline-delete-inside-leading.

Editable modes (unlocked, sdtLocked) delete one character per press from the
respective edge, wrapper preserved; content-locked modes (contentLocked,
sdtContentLocked) are blocked with content unchanged. All four lock modes
exercised per scenario. No divergence here - SuperDoc matches Word. No raw
Word data committed.
@caio-pizzol caio-pizzol requested a review from a team as a code owner May 28, 2026 23:36
@linear-code
Copy link
Copy Markdown

linear-code Bot commented May 28, 2026

SD-3237

SD-3218

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 2 files

Re-trigger cubic

…three

The contracts are step-based; assert content length original-i and wrapper
preserved after each press rather than only the final state.
@caio-pizzol caio-pizzol merged commit ebe43cb into main May 28, 2026
24 checks passed
@caio-pizzol caio-pizzol deleted the caio/sdt-inline-inside-edge-parity branch May 28, 2026 23:52
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants