Skip to content
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

Performance on Surge Website is Not Satisfactory #83

Open
ayanahye opened this issue Oct 23, 2024 · 0 comments
Open

Performance on Surge Website is Not Satisfactory #83

ayanahye opened this issue Oct 23, 2024 · 0 comments
Assignees

Comments

@ayanahye
Copy link
Member

ayanahye commented Oct 23, 2024

Problem Or Reason

As shown in the deployment logs, the performance on the website is not great. The time for the website to become fully interactive is 14.7 seconds which is quite long. The speed index is 8.3 seconds, so although the initial content loads somewhat quickly, the rest of the content (mostly the images) is taking a long time to show up. There is also a 810 ms delay in responding to user actions which is definitely a performance issue.


Proposed Solution

The main suggestion is to improve the rendering of images. We can do this by serving the images in formats such as WebP and AVIF and to also properly size the images. Another possible improvement is to reduce unused JavaScript and ensure Next.js components are used rather than React.js components where possible (like using <Image> component for lazy loading).


@ayanahye ayanahye self-assigned this Oct 23, 2024
ChrisFong604 added a commit that referenced this issue Dec 17, 2024
## Describe your changes
- Changed all img tags to next.js images
- Change all image formats to webp for more efficient rendering
- Remove unnecessary code

## Testing steps 
- Frontend testing

## Issue ticket number and link
- Issue #83. 

## Checklist before requesting a review
- [x] I have performed a self-review of my code
- [ ] If it is a core feature, I have added thorough tests.
- [ ] Do we need to implement analytics?

---------

Co-authored-by: ChrisFong604 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant