Page MenuHomePhabricator

[web] Simplify media gallery hooks
ClosedPublic

Authored by bartek on Fri, May 26, 1:26 AM.

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