⚡ Bolt: Optimize Zustand selectors and Konva rendering#5
⚡ Bolt: Optimize Zustand selectors and Konva rendering#5sourabhrajcodes wants to merge 1 commit into
Conversation
Refactored CanvasArea, Sidebar, and Toolbar components to use granular Zustand selectors instead of destructuring the state object. Added React.memo to ElementWrapper to prevent O(N) canvas re-renders. Co-authored-by: sourabhrajcodes <95199723+sourabhrajcodes@users.noreply.github.com>
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
💡 What: Refactored Zustand usage to use granular selectors instead of state destructuring, and wrapped
ElementWrapperinReact.memo(). Added inline documentation for these changes.🎯 Why: Destructuring the entire Zustand state object caused components to re-render unnecessarily when unrelated state changed. Rendering Konva components without memoization caused O(N) re-renders for every canvas element update.
📊 Impact: Eliminates unnecessary re-renders of the Sidebar, Toolbar, and CanvasArea components. Drastically reduces CPU time and frame drops during drag/drop or selection in documents with many elements.
🔬 Measurement: Use React DevTools Profiler to observe component renders. Unrelated state changes no longer trigger renders in these components, and moving a single element no longer triggers a render for all other elements on the canvas.
PR created automatically by Jules for task 17497330088753774832 started by @sourabhrajcodes