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

Using SASS pre-processor for css

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


The things that make me take a bit time is about sizing width and height

Community feedback

Adriano 34,090

@AdrianoEscarabote

Posted

Hello Obhasa Tanto Yangkasa Putra, how are you?

I really liked the result of your project, but I have some tips that I think you will like:

1- Document should have one main landmark, you could have put all the content inside the main tag click here

2- Page must contain a level 1 header click here

I noticed that you only used divs in this challenge, I recommend you to start using html 5 semantic elements.

The rest is great!!

Hope it helps...👍

Marked as helpful

1

@obhasa12

Posted

@AdrianoEscarabote Hi Adriano, thanks for the reference and the feedback. I will follow the tips that you gave me. I will make it better in the future.

1
Adriano 34,090

@AdrianoEscarabote

Posted

@obhasa12 no problem, keep coding!

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