From 77ec00df86b765da46a4bbbd6741fb1c68526e04 Mon Sep 17 00:00:00 2001 From: Jan-Henrik Bruhn Date: Sat, 27 Dec 2025 17:11:31 +0100 Subject: [PATCH] fix: Remove ineffective React.memo from PatternCanvas MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Address Copilot review feedback: PatternCanvas doesn't accept any props, so React.memo has no effect. The component re-renders are driven by Zustand store subscriptions which trigger regardless of memoization. Keep React.memo on PatternLayer since it does receive props and benefits from memoization. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Sonnet 4.5 --- src/components/PatternCanvas/PatternCanvas.tsx | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/components/PatternCanvas/PatternCanvas.tsx b/src/components/PatternCanvas/PatternCanvas.tsx index dbddabf..fae8a58 100644 --- a/src/components/PatternCanvas/PatternCanvas.tsx +++ b/src/components/PatternCanvas/PatternCanvas.tsx @@ -1,4 +1,4 @@ -import { useRef, useMemo, memo } from "react"; +import { useRef, useMemo } from "react"; import { useShallow } from "zustand/react/shallow"; import { useMachineStore, @@ -23,7 +23,7 @@ import { ZoomControls } from "./ZoomControls"; import { PatternLayer } from "./PatternLayer"; import { useCanvasViewport, usePatternTransform } from "@/hooks"; -export const PatternCanvas = memo(function PatternCanvas() { +export function PatternCanvas() { // Machine store const { sewingProgress, machineInfo } = useMachineStore( useShallow((state) => ({ @@ -277,4 +277,4 @@ export const PatternCanvas = memo(function PatternCanvas() { ); -}); +}