search

remix-run / remix-website

322 stars 72 forks source link

Showcase page crashes with too many examples #273

Closed brookslybrand closed 2 months ago

brookslybrand commented 2 months ago

See #272 for initial quick fix

The video switching is honestly a bit buggier looking than I would like it to be on mobile, so that either needs to be reworked, or we just need to get rid of it entirely.

From my initial testing, its seems like the problem is also due to the number of resources. We need to look into if we're properly loading them or not.

nikodunk commented 2 months ago

FWIW, just checked the image-only version of the showcase currently deployed on iOS, and it feels calmer to me. Also, the videos take a long time to make, tweak, edit to the right aspect ratio, export, get to seamlessly transition, etc. Especially if you get feedback like I did and need to RE-create them. May be more scalable to have more traditional static images mid-term anyway. Less is sometimes more ;)

brookslybrand commented 2 months ago

Yeah @nikodunk, I tend to agree. It was fun building out the video version, but I'm not sure it results in a better UX

brookslybrand commented 2 months ago

I do still want the videos for desktop @nikodunk, so not sure we're getting away from the tediousness of creating the examples 😅

Either way, I do think the mobile version needed to go, and I think I've resolved the underlying issue. Looking at #256 again now 👀