Page MenuHomePhabricator

[web] Simplify media gallery hooks
ClosedPublic

Authored by bartek on May 26 2023, 1:26 AM.
Tags
None
Referenced Files
Unknown Object (File)
Feb 21 2024, 11:39 AM
Unknown Object (File)
Feb 21 2024, 7:08 AM
Unknown Object (File)
Feb 21 2024, 6:52 AM
Unknown Object (File)
Feb 20 2024, 8:22 PM
Unknown Object (File)
Jan 10 2024, 10:22 PM
Unknown Object (File)
Dec 16 2023, 6:50 AM
Unknown Object (File)
Dec 16 2023, 5:15 AM
Unknown Object (File)
Nov 21 2023, 10:08 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
Lint Not Applicable
Unit
Tests Not Applicable