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

Order Summary Card || Html && Css

CyrusKabir 1,885

@CyrusKabir

Desktop design screenshot for the Order summary component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

P
Fluffy Kas 7,735

@FluffyKas

Posted

Hi, it looks really very good! Your CSS looks very neat and easy to understand, I'd only argue about your html. Not sure, if using sections here is appropriate. Also, heading should only increase by 1. And instead of a div (wrapper) you could perhaps use main. There's also no need to add a class name for the body as there's only one in a html document.

Marked as helpful

1

CyrusKabir 1,885

@CyrusKabir

Posted

@FluffyKas hello hello thank you for your feedback but i change right now i mean 2 minute ago that div and main problem and i forget use landmarks but thank for your help to improve myself (sorry if my english grammer is bad)

1

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