Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

HTML CSS then Responsive with div

@FORTUNE21

Desktop design screenshot for the Order summary component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


my code was simple.

Community feedback

Nerdytoo 30

@olalemi

Posted

  1. body{ background-image: url(images/pattern-background-desktop.svg); background-repeat: no-repeat; background-size: in pixels; height: 100vh; background-color: #E0E8FF;

  2. For responsiveness lookup media query via https://www.w3schools.com/css/css3_mediaqueries.asp

  3. your margin for your content should be defined margin-top: what is applicable px;

Hope this helps

0

@FORTUNE21

Posted

@olalemi Thank you for providing those useful tips. I have done the needful. All fit well now.

0
Nerdytoo 30

@olalemi

Posted

Nice work, but I think you can improve on these areas

  1. Your Final design was not responsive.
  2. Your background was disproportionate, I looked at your code, your background size should not be cover
  3. You didn't give room for your margin-top, it's hitting the top of my screen

Hope this helps

0

@FORTUNE21

Posted

@olalemi Thank you for your feedback. Can you help provide the appropriate code snippet to use for the stated areas to improve on.

Thank you!

0

Please log in to post a comment

Log in with GitHub
Discord logo

Join 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