Page MenuHomePhabricator

[web] Simplify media gallery hooks
ClosedPublic

Authored by bartek on May 26 2023, 1:26 AM.
Tags
None
Referenced Files
F3405287: D7996.diff
Tue, Dec 3, 7:52 PM
Unknown Object (File)
Wed, Nov 13, 1:55 PM
Unknown Object (File)
Wed, Nov 13, 1:55 PM
Unknown Object (File)
Wed, Nov 13, 1:55 PM
Unknown Object (File)
Wed, Nov 13, 1:53 PM
Unknown Object (File)
Oct 15 2024, 7:57 AM
Unknown Object (File)
Oct 13 2024, 5:41 PM
Unknown Object (File)
Oct 13 2024, 5:41 PM
Subscribers

Details

Summary

I noticed that the previous gallery logic had a dependency chain of useMemo hooks:
filteredMediaInfos -> mediaCoverPhotos -> mediaGalleryItems. This might be inefficient. If one dependency changes, all of them are fired.
Here is the React docs section on hook chains (with useEffect examples): https://react.dev/learn/you-might-not-need-an-effect#chains-of-computations

Merged all of these into one hook.

Depends on D7995

Test Plan

Media gallery works as before (repeat test plan from D7995)

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage