Skip to content

fix: usePress and useMove global event attachment timing#9749

Open
snowystinger wants to merge 5 commits intomainfrom
fix-9744-compiler-issue
Open

fix: usePress and useMove global event attachment timing#9749
snowystinger wants to merge 5 commits intomainfrom
fix-9744-compiler-issue

Conversation

@snowystinger
Copy link
Member

@snowystinger snowystinger commented Mar 6, 2026

Closes #9744

Applies the same logic to usePress, the other hook where I had done a similar transformation

✅ Pull Request Checklist:

  • Included link to corresponding React Spectrum GitHub Issue.
  • Added/updated unit tests and storybook for this change (for new code or code which already has tests).
  • Filled out test instructions.
  • Updated documentation (if it already exists for this component).
  • Looked at the Accessibility Practices for this feature - Aria Practices

📝 Test Instructions:

🧢 Your Project:

@github-actions github-actions bot added the RAC label Mar 6, 2026
@snowystinger snowystinger marked this pull request as ready for review March 16, 2026 03:55
@rspbot
Copy link

rspbot commented Mar 16, 2026

@devongovett devongovett changed the title fix: 9744 compiler lint issue fix: usePress and useMove global event attachment timing Mar 17, 2026
@rspbot
Copy link

rspbot commented Mar 17, 2026

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.

useMove refactor to useLayoutEffect breaks Playwright/synthetic drag events on Slider

3 participants