Brendan/animation 2 #57
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
manolingam
left a comment
manolingam
left a comment
There was a problem hiding this comment.
The deploy is successful and is looking good but there's a few more things that needs to be fixed.
-
The banner image in manifesto is not centered when going below large screen resolutions. It should be centered.
-
Also, the main hero banner doesn't appear for mobile screens & is also off the layout when on tab kind of resolutions.
-
There are a few empty files in the pr which needs to be removed. For example,
animation/base/displace.jsis empty. -
And in the
public/animation/imgfolder, I think there are images that are not used for the animation. Pls remove images that are not used for the animation. For example, our logo isn't animated right? But there are imgs related to it & a few others. -
Finally, can you remove the
.vscodefolder in the pr & add it to.gitignore.
Thank you :))
…to brendan/animation-2
|
@manolingam Thanks for your review. I've addressed everything and its all reflected here in the PR. |
No description provided.