Skip to content

fix: 添加组件卸载后的清理逻辑 #75

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
benym merged 1 commit into from
Mar 15, 2026
Merged

fix: 添加组件卸载后的清理逻辑 #75

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 3 additions & 1 deletion docs/rpamis-security-docs/src/components/PixelBlast.jsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -498,6 +498,8 @@ const PixelBlast = ({
});
let raf = 0;
const animate = () => {
// 组件卸载后 cleanup 会清空 threeRef，避免在已 dispose 的 renderer 上调用 render
if (!threeRef.current) return;
if (autoPauseOffscreen && !visibilityRef.current.visible) {
raf = requestAnimationFrame(animate);
return;
Expand Down Expand Up @@ -563,6 +565,7 @@ const PixelBlast = ({
if (threeRef.current && mustReinit) return;
if (!threeRef.current) return;
const t = threeRef.current;
threeRef.current = null; // 先清空，防止尚未取消的 requestAnimationFrame 再次执行 animate 时使用已 dispose 的对象
t.resizeObserver?.disconnect();
cancelAnimationFrame(t.raf);
t.quad?.geometry.dispose();
Expand All @@ -571,7 +574,6 @@ const PixelBlast = ({
t.renderer.dispose();
t.renderer.forceContextLoss();
if (t.renderer.domElement.parentElement === container) container.removeChild(t.renderer.domElement);
threeRef.current = null;
};
}, [
antialias,
Expand Down
Loading