-
Notifications
You must be signed in to change notification settings - Fork 23
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
[fix] remove lazy-loading for the images on above-the-fold area and remove CSS aspect-ratio and add transitions to lazy-loaded images #31
Conversation
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/nuxt-movies/nuxt-movies/npkc4e0fn |
@pi0 @addyosmani I'm going to remove the aspect ratio by the original CSS as we have |
b6efa82
to
ad1c603
Compare
b216caf
to
825e784
Compare
825e784
to
3ae7fd8
Compare
3ae7fd8
to
c4df7f1
Compare
40f9f6a
to
f39f25c
Compare
I'm going to go ahead and merge this PR although we have not solved the following issues: |
Thanks for merging @anton-karlovskiy was going to need it :D |
I think we should not lazy-load the images which are within the above-the-fold area for UX.