Skip to content

Comments

[pull] main from motiondivision:main#188

Merged
pull[bot] merged 24 commits intocode:mainfrom
motiondivision:main
Jan 6, 2026
Merged

[pull] main from motiondivision:main#188
pull[bot] merged 24 commits intocode:mainfrom
motiondivision:main

Conversation

@pull
Copy link

@pull pull bot commented Jan 6, 2026

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

mattgperry and others added 24 commits September 24, 2025 14:30
fix: Ensure useMotionValueEvent animationComplete fires for useSpring
Fixing React 19 cleanup when external ref returns cleanup function
In React 19, element.ref was removed and refs are now passed as regular
props. This change updates PopChild to check props.ref first (React 19)
and fall back to element.ref (React 18) for backwards compatibility.

Fixes #3428

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Fix React 19 ref compatibility in AnimatePresence popLayout mode
Only initialize the matchMedia listener for prefers-reduced-motion
when the reducedMotionConfig is not explicitly set to "never" or
"always". This avoids adding unnecessary event listeners when the
motion preference is statically configured.

Related to: #3304, #2444

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Add tests to verify:
- Listener is NOT initialized when reducedMotionConfig is 'never'
- Listener is NOT initialized when reducedMotionConfig is 'always'
- Listener IS initialized when reducedMotionConfig is 'user'
- Default config ('never') does not initialize listener
- Listener is only initialized once across multiple components

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
fix: optimize prefers-reduced-motion listener initialization
Prevents AnimatePresence from processing exit animations multiple times
when receiving rapid sequential events (e.g., Radix UI dismissable layers).
Adds a Set to track components currently processing exit and guards against
duplicate onExit calls.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Fix duplicate exit animation processing in AnimatePresence
@pull pull bot locked and limited conversation to collaborators Jan 6, 2026
@pull pull bot added the ⤵️ pull label Jan 6, 2026
@pull pull bot merged commit e82a97c into code:main Jan 6, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants