-
-
Notifications
You must be signed in to change notification settings - Fork 779
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
better documentation #391
better documentation #391
Conversation
README.md
Outdated
|
||
> 1) React window improves the responsiveness of your user interface by passing only a small portion of your data to the React reconciler. | ||
> 2) React window reduces the memory footprint of your application when displaying large datasets and prevents out of memory errors. | ||
> 3) React window reduces the need for your web services to support a paging interface. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This doesn't sound quite accurate. Windowing and data fetching are kind of separate issues, and often need to both be done (using a helper lib like react-window-infinite-loader
)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It worked for us since are components were so large. But I updated the pull request and removed the last sentence.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks~
I think it would be helpful to have a couple of sentences in the beginning better describing what it does.