Skip to content

test(app): keep Heading focus when the Heading Level changes - #14

Merged
rotespferd merged 1 commit into
mainfrom
chore/heading-level-focus-test
Oct 17, 2026
Merged

test(app): keep Heading focus when the Heading Level changes#14
rotespferd merged 1 commit into
mainfrom
chore/heading-level-focus-test

Conversation

@rotespferd

@rotespferd rotespferd commented Oct 17, 2026

Copy link
Copy Markdown
Contributor

Summary

Heading view
  rerender with level 1 -> 2
    same element, still focused, aria-level 2
Heading properties
  current Heading Level is aria-pressed
  click 3 -> onChange({ ..., level: 3 })

Regression test for the remount that stole focus when the Heading switched between h1/h2/h3 (fixed in #12).

Evidence

  • Before: no test covered Heading Level changes.
    After: pnpm test passes (16 tests). Not checked against the old h1/h2/h3 implementation.

Merge Danger

Door: two-way

Blast Radius: Tests only

🤖 Generated with Claude Code

@rotespferd @claude
test(app): keep Heading focus when the Heading Level changes
ebb8f46 
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@rotespferd
rotespferd merged commit 3cef2a7 into main Oct 17, 2026
1 check passed
@rotespferd rotespferd added the maintenance Refactoring, build, CI and tests label Oct 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

maintenance Refactoring, build, CI and tests

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@rotespferd