-
Notifications
You must be signed in to change notification settings - Fork 0
Open
Labels
frontendFrontend functionalityFrontend functionalityimplementationPossible implementation idea to keep in mindPossible implementation idea to keep in mindlow-priorityMay not be worked onMay not be worked on
Description
If performance ever becomes a problem, here are some steps that should probably be tried first:
- Wrap most components that need it in
React.memo(as described by this post, React doesn't memoize with shallow props comparison by default like I thought) - Split up context objects like
{ modificationState, setModificationState, loadPlaylist }into separate contexts, and maybe adding a convenience wrapper to avoid nested Providers in the JSX (reference)
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
frontendFrontend functionalityFrontend functionalityimplementationPossible implementation idea to keep in mindPossible implementation idea to keep in mindlow-priorityMay not be worked onMay not be worked on