Skip to content
This repository has been archived by the owner on Jan 2, 2025. It is now read-only.

frontend perf: first try #1664

Merged
merged 18 commits into from
Mar 7, 2024
Merged

frontend perf: first try #1664

merged 18 commits into from
Mar 7, 2024

Conversation

horacioh
Copy link
Collaborator

this is what I did so far:

  1. increate refetchIntervals on all the places we had one. now the smallest I believe is 10_000ms
  2. change queryClient networkMode so the app will work offline
  3. reorganize root and main components
  4. remove all interactivity (hovers) on publication content inside feed items
  5. memoize all list items

from what I could measure, this changes make the app work "a little faster" and prevents a lot of computation on interactions.

feel free to add any suggestions. if we as a team believe this is not important, I'm ok with trashing this away.

@horacioh horacioh self-assigned this Feb 23, 2024
@horacioh horacioh force-pushed the app-perf branch 2 times, most recently from 8a0e756 to 6b567b7 Compare March 4, 2024 22:19
@horacioh horacioh merged commit ce9b97c into main Mar 7, 2024
1 check failed
@horacioh horacioh deleted the app-perf branch March 7, 2024 09:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants