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

Use Html and pure Css clearly handling the bem methodology :)

@sergiopin22

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


What can I improve on the receptive design friends?

Community feedback

@pikapikamart

Posted

Hey, great work on this. Layout seems fine but you are missing the background image, the wave image one. Perhaps you forgot to add it?

Some suggestions would be:

  • If you inspect your website in dev tools at the bottom, you can see that your layout is being squished. This is caused by the height: 100vh on your main tag. If you set an element to have height: 100vh it might look fine to your monitor/screen, but if you scale down your browser's height, you can see the distortions. This is because using height: 100vh only occupies the current viewport's height. Removing this will be really good or you can set a max-height: 100vh instead of height: 100vh. Also on your body tag, do not set the font-size: 2vh , do not use scaling units on font-sizes unless you are using a clamp function. Use rem units
  • Good choice on using article!
  • your page lacks h1 element, you can use the h1 to wrap the order summary text instead of using h2 on it.
  • Your proceed to payment and cancel order html element is not suited. You used div which is not accessible at all. You can use either a tag or button on those two elements. If you think that the functionality links you up in another page, use a tag. If you think that clicking those element pops up a modal or some sort, use button.
  • Also you don't have to position: absolute the attribution.

Aside from those, good job.

Marked as helpful

1

@sergiopin22

Posted

Hey really, thank you very much, what a good feedback you have given me, I really appreciate you, I hope to continue practicing I am from Colombia and I have to translate your opinion lol

Greetings what a good level I see that you drive :)

0
Yunus Alif 150

@frontendnus

Posted

I only have a little bit feedback. I think you overuse flexbox there where we can still do without flexbox. But your project looks good for me

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