Skip to content

Remove website background video on mobile #1799

@aaronmondal

Description

@aaronmondal
Contributor

The background video on our mobile landing page is a fairly large download. It causes inconsistent performance, sometimes dropping lighthouse to below 80 due to an LCP of over 5 seconds 😱.

It seems like a good idea to just remove on mobile it as we probably shouldn't burden users on mobile networks with a 1,597.6 KiB download for a video that is barely visible.

@Harsh9485 @shamun-khatri Would one of you be interested in tackling this? @Evaan2001 Is already preparing a new video for #1784.

Activity

Harsh9485

Harsh9485 commented on May 25, 2025

@Harsh9485
Contributor

I’d be happy to work on this!

It looks like a good quick issue, but for me it’s also a great way to explore and understand more of this complex project. I really enjoy learning through contributions like this — they help me grow and get more comfortable with the codebase.

changed the issue type fromtoon May 25, 2025
Harsh9485

Harsh9485 commented on May 27, 2025

@Harsh9485
Contributor

I removed the video for mobile devices, and it seems to work fine locally.
However, when I tried to analyze the performance using Lighthouse, it got stuck for over 30 minutes.

@aaronmondal What do you think I should do in this case?

Harsh9485

Harsh9485 commented on May 27, 2025

@Harsh9485
Contributor
Screen.Recording.2025-05-27.101645.mp4

Is it alright if I just open a pull request and you take it over from there?

Evaan2001

Evaan2001 commented on May 27, 2025

@Evaan2001
Contributor

@Harsh9485, thanks for taking this on :). I think what @aaronmondal meant by the background video is the animation we see behind "The Parallel Compute Platform", not the ~2 mins long walkthrough video. Think you can make this change?

Harsh9485

Harsh9485 commented on May 28, 2025

@Harsh9485
Contributor

@Evaan2001 Thank you for pointing out my mistake.

In issue #1784, we discussed that the video resolution is very low.
@aaronmondal mentioned that removing the background video might be a good idea, and I mistakenly thought he was referring to the main walkthrough video.

I accept my mistake, and I’ll go ahead and fix it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

Type

Projects

No projects

Milestone

No milestone

Relationships

None yet

    Participants

    @aaronmondal@Evaan2001@Harsh9485

    Issue actions

      Remove website background video on mobile · Issue #1799 · TraceMachina/nativelink