fix: corrected the downed flag state to ensure the correct invocation of onChange()

pmndrs / drei

🥉 useful helpers for react-three-fiber

https://drei.pmnd.rs/

MIT License

7.83k stars 642 forks source link

fix: corrected the downed flag state to ensure the correct invocation of onChange() #1982

Open cincolunes opened 4 weeks ago

cincolunes commented 4 weeks ago

Why

Fixes #1980

What

Fix the behavior of the onChange() function to return the current states of the selected items instead of the previous states or an undelivered state. I've just added the imperative punctuation mark to the downed state within the useEffect function.

Checklist

[ ] Documentation updated (example)
[ ] Storybook entry added (example)
[x] Ready to be merged

No documentation or storybook changes, since this works just as expected

vercel[bot] commented 4 weeks ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
drei	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jun 28, 2024 2:17am

codesandbox-ci[bot] commented 4 weeks ago

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.