-
Notifications
You must be signed in to change notification settings - Fork 249
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
Flatlist API #4
Flatlist API #4
Conversation
RecyclerFlatList.tsx
Outdated
horizontal: boolean; | ||
} | ||
|
||
class RFlatList extends React.PureComponent<RFlatListProps> { |
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.
Is it possible to use a component instead of a class? 👀
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.
I'd keep it a class since functional components inherit from React.Component
instead of React.PureComponent
. You could achieve the purity with some HoC or by implementing shouldComponentUpdate
but that's not really worth it imho.
Additional resources: https://blog.logrocket.com/react-pure-components-functional/
* added experimental web support * Fixed inverted mode issue on web * updated known issues * added few tests * added sample for web * improved sample * removed local script * added height update logic * improved sample * fixed style issue * rephrase known issue * lint fix * updated e2e screenshot * fix broken e2e image * Added sample for CellRenderer and added support to disable autolayout * some updates to docs * Added render target support * data change won't rerender unless renderItem changes * added test * Fixed ref warning * review comments * review fixes * fixed script * build fix * added changelog * updated changelog * updated changelog * fixed type * Update CHANGELOG.md * lint fix Co-authored-by: Marek Fořt <marek.fort@shopify.com>
Draft PR for using
RecyclerListView
withFlatList
API.