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)
Thu, Apr 4, 9:10 PM
Unknown Object (File)
Thu, Apr 4, 9:10 PM
Unknown Object (File)
Thu, Apr 4, 9:09 PM
Unknown Object (File)
Thu, Apr 4, 9:01 PM
Unknown Object (File)
Sun, Mar 31, 10:08 AM
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
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