hitmorecode• 6,230
@hitmorecode
Posted
Nice well done, a few pointers
- You have the background image inside the
.container
move it to the body - Also
min-height: 100vh;
on the container, move it to the body
## Add this to the media query
body { /**/
background-image: url("images/bg-desktop.svg");
background-repeat: no-repeat;
background-size: cover;
}
- You gave Huddle a
h1
, but you set the display to none. Delete this and use it on the title<h1>Build The Community Your Fans Will Love</h1>
- On your html structure you have
<main>
inside a<div>
it should be the other way around.<main>
must be at the top.
1
Himanshu_2301• 320
@Himanshu-196018
Posted
@hitmorecode thank you for the feedback and I know this solution contains many errors.
I am experimenting with this project with some onload animations, but it didn't going as I am expecting it to go. Still working in a separate branch and will merge and resubmit the solution again.
I will also try to include all the key points you mention.
0