Skip to content

Fix detached element cleanup check#214

Merged
bartekplus merged 1 commit intomasterfrom
codex/fix-detached-element-cleanup
Feb 24, 2026
Merged

Fix detached element cleanup check#214
bartekplus merged 1 commit intomasterfrom
codex/fix-detached-element-cleanup

Conversation

@bartekplus
Copy link
Owner

@bartekplus bartekplus commented Feb 24, 2026

Summary

  • fix isInDocument to use document.contains(element) instead of ownerDocument === document
  • prevent detached elements from being treated as still attached
  • add regression test for detached/attached/removed lifecycle

Validation

  • npm test -- tests/utils.test.ts

@bartekplus bartekplus merged commit 480f22b into master Feb 24, 2026
4 checks passed
@bartekplus bartekplus deleted the codex/fix-detached-element-cleanup branch February 24, 2026 16:49
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