javascript/rive-react
1
0
Fork 0
mirror of https://github.com/rive-app/rive-react.git synced 2026-03-13 08:22:30 +08:00
Code Issues Packages Projects Releases Wiki Activity

fix: unify effects in useViewModelInstance

Browse Source
This commit is contained in:
Adam
2025-04-29 17:18:34 -07:00
parent d9e61373b3
commit 384b62c1b7
1 changed files with 4 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
src/hooks/useViewModelInstance.ts
View File

@@ -54,12 +54,9 @@ export default function useViewModelInstance(
const areParamsChanged = !areParamsEqual(paramsRef.current, params);
shouldUpdate.current = isViewModelChanged || areParamsChanged;
viewModelRef.current = viewModel;
paramsRef.current = params;
}, [viewModel, name, useDefault, useNew]);
useEffect(() => {
if (!shouldUpdate.current && instanceRef.current) {
return;
}
@@ -91,15 +88,12 @@ export default function useViewModelInstance(
instanceRef.current = result;
setInstance(result);
shouldUpdate.current = false;
}, [viewModel, name, useDefault, useNew]);
// Automatically bind to Rive when requested, if not already bound
useEffect(() => {
if (!rive || !instance) return;
if (rive.viewModelInstance !== instance) {
rive.bindViewModelInstance(instance);
// Bind instance to Rive if needed
if (rive && result && rive.viewModelInstance !== result) {
rive.bindViewModelInstance(result);
}
}, [rive, instance]);
}, [viewModel, name, useDefault, useNew, rive]);
return instance;
}