Design comparison
SolutionDesign
Community feedback
- @satyammjhaPosted over 2 years ago
Nice attempt @Arearda, I have a few suggestions for you to further improve your code:-
- Your colors are too saturated for colors please see the style guide of the projects
- Use the
<br>
tag after Gabrielle and Eau, it will break the line of the sentence. - Increasing the letter spacing of PERFUME by using
letter-spacing: 2px
. - Move the button bit lower so that it will cover the more empty space.
- use
cursor: pointer
for the button. - Change the background color of the button on hover by using
button:hover{background-color: hsl(158, 35%, 29%);}
- Old price is not aligned correctly it's below the final price. Please make it horizontally middle of the new price.
Hope these points will help you. Happy Coding 🤗✌🏻.
0 - @afaiz-spacePosted over 2 years ago
Hey @Arearda,
- Replace
height:100vh
withmin-height:100vh
in the body element. - also add media query in the project.
- change the button and price color.
0 - Replace
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord