heyxyz / hey

Hey is a decentralized and permissionless social media app built with Lens Protocol 🌿
https://hey.xyz
GNU Affero General Public License v3.0
23.37k stars 1.55k forks source link

Add virtualized list support for feed, notifications and users list so we will not overload the DOM usage #837

Closed bigint closed 1 year ago

benbaessler commented 1 year ago

@bigint Is this not already integrated with the use of react-infinite-scroll-component library? For example here.

bigint commented 1 year ago

@benbaessler They don't have virtual list support 🙇🏼

benbaessler commented 1 year ago

@bigint wdyt about using a library like react-window (link here)? I'll try to integrate it and see if it works as intended.

bigint commented 1 year ago

Yup i'm really happy if we are using react-window, aka one of my favorite one

yyj2000 commented 1 year ago

~~

bigint commented 1 year ago

$250 USD Bounty https://app.dework.xyz/lenster-23720/lenster?taskId=31e9b837-5f6e-4673-8262-dba88309f0a6

rahuldkjain commented 1 year ago

hey @bigint, I would to like to work on this enhancement

bigint commented 1 year ago

Amazing @rahuldkjain 🤝 you can go ahead and submit your work here when done

https://app.dework.xyz/lenster-23720/lenster?taskId=31e9b837-5f6e-4673-8262-dba88309f0a6

chngln commented 1 year ago

:) i can try it

ManyRios commented 1 year ago

@bigint This is already implemented with react-virtuoso?

bigint commented 1 year ago

@ManyRios yes we dont it in all places except feed, closing this issue now!

github-actions[bot] commented 9 months ago

This issue has been locked since it has been closed for more than 10 days.

If you found a concrete bug or regression related to it, please open a new bug report.