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
Expandable item #92
Comments
Can you please point me at a repro that shows what's not working for you? Here's a Code Sandbox you can fork to get started. |
I've added expandable items to Brian's example and it works: https://codesandbox.io/s/kmzrylpx7o |
I'm going to close this for now. Will re-open if repro steps are provided. 😄 |
I know this is closed for now, but what about an animated expanding div. Not just something with 2 hard coded values. I have a need to use the |
Thanks for this, it helped me a bunch |
@fonty422 Have you ever found a solution for animated expanding divs? |
@itayganor I ended up using a different package (react-virtuoso) and going through the less painful (but still painful) method of providing a pre-culled list of items. Also, the material-ui page discussed it as a recommendation (along with react-window) for large lists. The demo page will show you if it's any good for your purposes. |
@fonty422 This is amazing. So simple. Thank you! I actually have a question but I don't want to spam this discussion, so I'm creating it over there. |
@fonty422 @itayganor do you have sample code of how you implemented nested lists using react-virtuoso? I'm not able to find examples of how it is done. |
@fonty422 Thank you for bringing my attention to this package. It worked perfectly for my team's use case. Simple and easy to implement. |
You're welcome. It's still my preferred package for this work for exactly this reason. |
I am working on smaller list where we need an option to expand item (and change height of it). It seems like current api doesn't support that.
resetAfterIndex
is causing re-rendering, but not refreshing the height of the actual item. Doesreact-window
currently support this option?The text was updated successfully, but these errors were encountered: