ppy / osu-web

the browser-facing portion of osu!
https://osu.ppy.sh
GNU Affero General Public License v3.0
978 stars 382 forks source link

Beatmap listings should be unloaded at some point #2117

Closed Xinnoh closed 6 years ago

Xinnoh commented 6 years ago

Same issue as #2048 but on the beatmap listing

imo having 40+ maps per page like the old system would work better because there's no correlation between maps, whereas forum posts directly affect each other.

peppy commented 6 years ago

Pagination is not coming back, but unloading is a valid concern.

Xinnoh commented 6 years ago

I think this one should be required for live, it would be impossible to browse many maps without it

peppy commented 6 years ago

We should either limit number of paginations possible or indeed do some kind of unload/reload magic.

notbakaneko commented 6 years ago

I'm going to see if we can use react-virtualized or something for this, while I'm looking at a bunch of stuff related to this page

notbakaneko commented 6 years ago

runs fast even with 10000 items loaded ᕕ( ᐛ )ᕗ