HomePhabricator
Diffusion Comm fc48c1d48bfe

[web] Memoize `NotificationsOption` `onSelect` prop w/ `useCallback` hook

Description

[web] Memoize NotificationsOption onSelect prop w/ useCallback hook

Summary: The NotificationsOption onSelect callback prop was being defined inline instead of being memoized with React.useCallback. This diff just fixes that.

Test Plan: Clicked through the NotificationsModal and things worked as expected.

Reviewers: def-au1t, palys-swm, benschac, yayabosh

Reviewed By: palys-swm, yayabosh

Subscribers: ashoat, Adrian, yayabosh

Differential Revision: https://phabricator.ashoat.com/D4042

Details

Provenance
atulAuthored on May 16 2022, 10:54 AM
Reviewer
tomek
Differential Revision
D4042: [web] Memoize `NotificationsOption` `onSelect` prop w/ `useCallback` hook
Parents
rCOMMe26f1c0f5b09: [iOS] Fix `CryptoTest.mm`
Branches
Unknown
Tags
Unknown