andywer / andywer/use-inline-memo

deps default to []

Open
#4 4 comments 0 reactions 0 assignees View on GitHub
enhancement help wanted question
Dominant language
TypeScript
Stars
164
Forks
3
Avg merge
17h 56m
Merged PRs (30d)
2

Description

Awesome package. Seriously, it should be part of React core!
Suggestion:
Instead of:
`memo.textChange(event => setValue(event.target.value), [])`
accepts this:
`memo.textChange(event => setValue(event.target.value))`

Question: How can you measure that the overhead in memo is faster then a component rerender that justify the use of it?

Great work!

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.