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

Lazy loading is implement #130

Closed
wants to merge 2 commits into from
Closed

Lazy loading is implement #130

wants to merge 2 commits into from

Conversation

shraddha761
Copy link

Resolves #123

Description

Screenshots

Checklist

  • My code adheres to the established style guidelines of the project.
  • I have included comments in areas that may be difficult to understand.
  • My changes have not introduced any new warnings.
  • I have conducted a self-review of my code.

@vercel
Copy link

vercel bot commented Jul 4, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
jarvis ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 27, 2023 1:25pm

@shraddha761
Copy link
Author

@Abhijit25Mishra Please review my pr now

Copy link

@Abhijit25Mishra Abhijit25Mishra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

seems fine. Please add some images showing the performance improvements. (you can find these details in inspect element tabs.)

@shraddha761
Copy link
Author

shraddha761 commented Jul 5, 2023

@Abhijit25Mishra Please have a look

Screenshot 2023-07-05 193305

@Abhijit25Mishra
Copy link

Abhijit25Mishra commented Jul 5, 2023

@shraddha761 try showing the difference with and without the changes you made, make note of the stats and observations and write in detail about it.

this will actually help the site a lot (from what i know about lazy loading), just try and show some stats to prove it.

@J0SAL J0SAL added level3 and removed level2 labels Jul 18, 2023
@shraddha761
Copy link
Author

@J0SAL please review I have added loading to website

@J0SAL
Copy link
Owner

J0SAL commented Jul 22, 2023

@Abhijit25Mishra can you review

@Abhijit25Mishra
Copy link

@shraddha761 can you implement it such that only the first time the user visits this page, it will load this animation. you can try using caching for that.

also i see that the image gets loaded first then it appers to rotate, this makes it look bad, what you can do is add opacity changes, what i mean is that the start opacity should be zero and at 0.25 second also it should be zero then from 0.25 second to 3 sec, it should gradually increse to 100%. i know it sounds complicated, but its really simple.

@shraddha761
Copy link
Author

@Abhijit25Mishra how to start opacity from 0.25 I tried using delay but it didn't work

and for loading for first time user i have used session storage

@Abhijit25Mishra
Copy link

Yeah I saw that session storage thing in code, I didnt really get why it didn't work.

Please check online about opacity, try reading about keyframes. Search on gpt and llaama

@shraddha761
Copy link
Author

@Abhijit25Mishra I have done some changes please review

@shraddha761
Copy link
Author

@J0SAL I didn't understand , you mean to do it from another way

@J0SAL
Copy link
Owner

J0SAL commented Aug 9, 2023

@J0SAL I didn't understand , you mean to do it from another way

wait, let me review it again

Copy link
Owner

@J0SAL J0SAL left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

After the user logs in the loading image is still there
image

Comment on lines +44 to +46
<Loading/>
<Container className="mt-5 py-3 bg-white">
<Suspense fallback={<div>Loading...</div>}>
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Instead of this can you directly put Loading inside Suspense?
like, <Suspense fallback={<Loading/>}>

@shraddha761
Copy link
Author

@J0SAL Please review now

@J0SAL
Copy link
Owner

J0SAL commented Aug 10, 2023

@shraddha761 there weren't any updates in the code

@J0SAL J0SAL closed this Aug 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Enable lazy loading to improve website performance
3 participants