In D8958 I should have memoized the bottom sheet handle component. This diff addresses that concern
Details
Details
- Reviewers
atul inka - Commits
- rCOMMe1b680de0bbe: [native] memoize BottomSheetHandle
confirmed that the handle still worked as expected
Diff Detail
Diff Detail
- Repository
- rCOMM Comm
- Lint
No Lint Coverage - Unit
No Test Coverage
Event Timeline
Comment Actions
Personal preference is memoization "inside" the component w/ useMemo hook so everything is self-contained and memoization is consistent and whatnot, but this is valid