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 Component

@CassiuSRSA

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


Any feedback will be greatly appreciated :D

Community feedback

Fidel Lim 2,775

@fidellim

Posted

Hi @CassiuSRSA,

Great job finishing the project! It looks great on desktop and mobile devices. Some suggestions I would like to share are of the following:

  • you can add some margin/padding to your component so that it can have some spacing on mobile view.
  • Heading tags are not in order. This is important for assistive technology users to navigate the website. If you have h3, make sure that h2 and h1 exists.

I hope these helps :) Well done.

Marked as helpful

1

@CassiuSRSA

Posted

@fidellim Thank you for your input, I have corrected the heading tags. Also added margin so it has some gaps on mobile view.

1

@mdajmalshadab

Posted

Hey nice work, you just need to add some background color to your page body so that it could match with the challenge template. Keep coding!😊

Marked as helpful

1

@CassiuSRSA

Posted

@mdajmalshadab thanks for the feedback, I eyeballed the background color think I made it too light. Will correct thanks

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