Skip to content

Version Packages#1802

Merged
stefcameron merged 1 commit intomasterfrom
changeset-release/master
Jan 29, 2026
Merged

Version Packages#1802
stefcameron merged 1 commit intomasterfrom
changeset-release/master

Conversation

@stefcameron
Copy link
Member

@stefcameron stefcameron commented Jan 29, 2026

This PR was opened by the Changesets release GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to master, this PR will be updated.

Releases

focus-trap-react@12.0.0

Major Changes

  • 763eae4: BREAKING: Updated focus-trap dependency to v8.0.0. The breaking change is that onPostActivate() is now correctly called after the initial focus node is focused (it was previously called before due to a bug with the initial focus delay). See the focus-trap changelog for more details.

Summary by CodeRabbit

  • Breaking Changes
    • Updated focus-trap dependency to v8.0.0. The onPostActivate() callback now fires after the initial focus node receives focus, as opposed to before in previous versions.
    • Version bumped to 12.0.0 to reflect these breaking changes.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link

coderabbitai bot commented Jan 29, 2026

Walkthrough

The PR bumps the package version to 12.0.0 and documents a breaking change stemming from a focus-trap dependency upgrade to v8.0.0. The onPostActivate() callback behavior has shifted to fire after the initial focus node is focused, rather than before. The changeset file is removed as part of the release process.

Changes

Cohort / File(s) Summary
Release & Documentation
package.json, CHANGELOG.md, .changeset/focus-trap-8-upgrade.md
Version bump to 12.0.0 with changelog entry documenting the breaking change in focus-trap v8.0.0 behavior. Changeset file removed post-merge.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Description check ❓ Inconclusive The PR description is auto-generated by the Changesets release action and lacks the detailed checklist items required by the template, but clearly documents the release details and breaking changes. While this is an automated release PR, consider filling out or adapting the PR checklist to confirm all required validations were performed before release (browser compatibility, test coverage, changelog updates, etc.).
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately reflects the main purpose of this PR—a version bump release triggered by the Changesets action, though it's somewhat generic.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@stefcameron stefcameron merged commit 932495b into master Jan 29, 2026
3 checks passed
@stefcameron stefcameron deleted the changeset-release/master branch January 29, 2026 16:53
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