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 responsive Pure CSS

@SamCastle16

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


Hello everyone!

This is my first attempt at doing this. I only have two months learning HTML and CSS, and obviously this will have errors, so I hope you will tell me what you think about my development of this mini-project and what things could be changed and improved. It would be greatly appreciated!

Community feedback

Burak CLN 385

@brkcln

Posted

Hello there, Thanks for submitting your project. You did well, 3 - 4 thing i can say..

  • avoid to set "height" not a good practice (most of time),
  • I see unnecessary codes in the codes you wrote. Maybe you wanna look at,
  • For the font size try to use "rem" instead of "em" [watch this] (https://www.youtube.com/watch?v=pautqDqa54I)
  • Can add "main",
  • Can add "container" better experince, good work, keep up :)

Marked as helpful

1

@SamCastle16

Posted

@brkcln Thank you very much for your comment Burak, that helps me a lot to improve. I will consider your advice.

In the case of "heights", which components of the code could be removed?

Again, thanks a lot!

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