docs (example/column-resize-performant): checking resize inside UseMe…#6126
docs (example/column-resize-performant): checking resize inside UseMe…#6126rezha4 wants to merge 1 commit intoTanStack:mainfrom
Conversation
…mo to avoid unneccessary remounts
|
WalkthroughThe memoization strategy for the TableBody component in the column-resizing performance example was restructured. The MemoizedTableBody now renders whenever enableMemo is true and uses a conditional comparator that activates memoization only during active column resizing, forcing re-renders outside of resizing sessions. Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes
Poem
Pre-merge checks and finishing touches❌ Failed checks (1 warning, 1 inconclusive)
✅ Passed checks (1 passed)
✨ Finishing touches
🧪 Generate unit tests (beta)
Tip 📝 Customizable high-level summaries are now available!You can now customize how CodeRabbit generates the high-level summary in your pull requests — including its content, structure, tone, and formatting.
Example:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
…mo to avoid unneccessary remounts
🎯 Changes
Unneccessary component remounts fixed by checking resize inside UseMemo.
Refer to issue and original solution #6121 by @ItaiYospehi
✅ Checklist
pnpm test:pr.🚀 Release Impact
Summary by CodeRabbit