From bba4db0f91d57ceb95cd7b8416bda67be0980be8 Mon Sep 17 00:00:00 2001 From: Tanish Manku Date: Wed, 27 Aug 2025 14:14:28 -0700 Subject: [PATCH] Set state in a useEffect to avoid react warning --- .../editor3d/app/src/pages/PageEdit/PageEdit.tsx | 13 ++++++++----- 1 file changed, 8 insertions(+), 5 deletions(-) diff --git a/frontend/apps/editor3d/app/src/pages/PageEdit/PageEdit.tsx b/frontend/apps/editor3d/app/src/pages/PageEdit/PageEdit.tsx index 83665ff922..6d0db0e9b1 100644 --- a/frontend/apps/editor3d/app/src/pages/PageEdit/PageEdit.tsx +++ b/frontend/apps/editor3d/app/src/pages/PageEdit/PageEdit.tsx @@ -249,11 +249,14 @@ const PageEdit = () => { //const supportsMaskedInpainting = modelConfig?.tags?.includes(ModelTag.MaskedInpainting) ?? false; const supportsMaskedInpainting = selectedImageModel?.usesInpaintingMask ?? false; - if (!supportsMaskedInpainting && (store.activeTool !== "select" || store.lineNodes.length > 0)) { - // TODO: Implement a new mode for unsupported masking and hide the nodes layer instead of clearing - store.setActiveTool("select"); - store.clearLineNodes(); - } + useEffect(() => { + if (!supportsMaskedInpainting && (store.activeTool !== "select" || store.lineNodes.length > 0)) { + // TODO: Implement a new mode for unsupported masking and hide the nodes layer instead of clearing + store.setActiveTool("select"); + store.clearLineNodes(); + } + }, [store, supportsMaskedInpainting]); + /* *