Looking to hire developers?
<link href="https://kit-free.fontawesome.com/releases/latest/css/free-v4-font-face.min.css" media="all" rel="stylesheet" id="font-awesome-5-kit-css"><link href="https://kit-free.fontawesome.com/releases/latest/css/free.min.css" media="all" rel="stylesheet" id="font-awesome-5-kit-css"><img src="images/logo.svg" class="logo"><img src="images/screen-mockups.svg" class="img-mockups container"><img class="icon" src="images/icon-communities.svg"><img class="icon" src="images/icon-messages.svg"><img src="images/illustration-grow-together.svg" class="stretch-width"><img src="images/illustration-flowing-conversation.svg" class="stretch-width"><img src="images/illustration-your-users.svg" class="stretch-width"><img src="images/icon-phone.svg"><img src="images/icon-email.svg"><input type="email">Any feedback? :)
@emestabillo
Posted
Hi @bluepersia, great job in completing this challenge! Just a few little details - the buttons are missing their box-shadows and it'd be nicer to add hover styles to both the buttons and links in the footer. The second and third main blocks are also not displaying their desktop version of the wave dividers. Hope this helps!
1
@bluepersia
Posted
@emestabillo Thank you very much, great feedback.
1
Looks really nice and clean in both desktop and mobile version! For some reason the link to your github repository isn't working so I couldn't look at your code. Just looking at the design though, the only suggestion I have is for the input box consider adding type="email" and required so that it doesn't let the user succeed in pressing the Subscribe button until they have entered a valid email address. Keep up the great work! -Jen
0
@bluepersia
Posted
@En-Jen Well spotted, thank you very much for the feedback.
0
Join over 180,000 people taking the challenges, talking about their code, helping each other, and chatting about all things front-end!
