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

Not Responsive

Ibraheemā€¢ 10

@ibraheemabdlhafeez

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


Please check it and tell problems i made

Community feedback

Vanza Setiaā€¢ 27,855

@vanzasetia

Posted

šŸ‘‹ Hi Ibraheem!

I agree with all feedback that Jason has said. In addition to that, I recommend you, to join the Frontend Mentor slack to get help.

Also if you're not comfortable yet with responsive website, I recommend you to learn about it first and then go back and refactor this project.

šŸ˜‰ Keep learning!

Marked as helpful

1

Ibraheemā€¢ 10

@ibraheemabdlhafeez

Posted

@vanzasetia Thank you, i will

0
Jason Heysā€¢ 175

@heyitsgany

Posted

As you've already mentioned, this website is not responsive. You'll need to use media queries to set the styling for mobile devices.

These are a few things to look into fixing with your design (although certainly not an exhaustive list).

  • Make sure you're using landmark elements (such as main, footer, nav, section) instead of always using a div. (You have used these a class declarations, so you know where they need to be).
  • Instead of pasting the code from the SVG images into the HTML, you can of course use an <img> tag to place the image. This should make your HTML more readable (however it doesn't break things if you don't do this).
  • Following on from this, you have placed the background image into the HTML, instead of using CSS to place the background. My advice would be to set the background on the body using the background CSS property.
  • Your CSS has very strict specificity. You normally don't need to use so many selectors to style an element. For example, you use ".container .card .body .about .details", where you could achieve the same styling just using the ".details" selector.

Keep working on it, and you'll get there!

1

Ibraheemā€¢ 10

@ibraheemabdlhafeez

Posted

@heyitsgany I'm very grateful :) thank you i will make sure to do your advice next time.

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