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-Compenent HTML+CSS

Dellenar 160

@dellenar

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 There,

It's been a month since i start learning how to code. If you have a time please review my code and let me know if something i should look into or if there is any method that I'm using wrong.

Thank you in advance !

Community feedback

@yannmarc

Posted

Hi Friend!

  • Your code looks pretty good and semantic in terms of HTML.
  • Your CSS Code can be improve greatly,
    • Using :root to define your CSS variables like colours, fonts, e.t.c
    • You can use or create your own utlity classes to avoid repeating yourself to speed up your development.

That's all what I got for you, Great job. Keep Up. Regard

Marked as helpful

0

Dellenar 160

@dellenar

Posted

@yannmarc

Hey,

Thank you so much for the feedback you just did. I will definitely use :root for the next structures i will do.

I already know that var() & :root funtcion but constantly forget about it. From now on i'll try to make it habit

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