fix: useTexture callback parameter and behavior

[jwueller]

Why

Fixes #1968
Fixes #1969

What

Fixes the behavior of the useTexture(..., onLoad) callback according to the declared types, as well as the conditions under which it is fired (i.e. when the data changes, not when the function does).

Checklist

• Documentation updated (example)
• Storybook entry added (example)
• Ready to be merged

No documentation or storybook change is required, because this just fixes previously-documented behavior.

[vercel]

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	May 30, 2024 6:35pm

[codesandbox-ci]

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.

[FarazzShaikh]

@jwueller Can you please resolve merge conflicts and merge master into this? There was a refactor to Drei. Please let me know if youre not able to, we will close this and reopen a new PR on latest master

[clementroche]

@FarazzShaikh the conflicts resolution is the only blocker?

[FarazzShaikh]

@clementroche yes. But the conflicts are extensive. Better to start a new PR with latest upstream

[clementroche]

omw @FarazzShaikh

[clementroche]

done in #2142 @FarazzShaikh

[abernier]

why another PR for solving conflicts? is this one to be considered as outdated/duplicate now?

[FarazzShaikh]

@abernier Yes, this one is duplicate now and should be closed. The last time I checked, there were a lot of conflicts as this PR was made before the refactor.

[abernier]

@abernier Yes, this one is duplicate now and should be closed. The last time I checked, there were a lot of conflicts as this PR was made before the refactor.

ok, closing as duplicates in favor of #2142