Huddle landing page with a single introductory section using HTML&CSS
Design comparison
Solution retrospective
Fun project Comments and Reviews can come in.
Thanks
Community feedback
- @HassiaiPosted almost 2 years ago
@Greg-davi, how are you? Well done in completing this challenge Instead using both external style-sheet and inline style use only in the style sheet in the body{ background-image: url(image.svg) ; background-repeat: no-image ; background-color: hsl(put in the values)} Use rem or em as unit for the margin and padding . for the font-size use rem . Give the .head a padding top and bottom: .head {padding: 1em 0;} no need for height the padding will be enough The main needs a margin-top and bottom value: main{ margin: 2em 0}. there is no need for flex-direction: row; that is the default value. after giving the margin top and bottom value to the main there is no need to give the write-up a margin left or margin top. give the write-up a padding top value. For space between the img and the write-up , you use gap in the main: main { display: flex; gap: 2rem;} or img + .write-up{margin-right: 2em}. I don't see the need for class wrapper and container using the main and the head should be enough. You forgot to set hover for the button and social. Aside from these, everything is good. HAPPY CODING.
Marked as helpful0@neena-coderPosted almost 2 years ago@Hassiai Thank you for your suggestions, I highly appreciate.
0
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