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

HTML AND BASIC CSS

Vishal 260

@vishal-singh5128

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 Suggestions or comments are appreciated !

Community feedback

Renzorr 870

@Renzorr

Posted

👾Hi @Vishal, congratulations on your solution!👋 Welcome to the Frontend Mentor Coding Community!

I’ve few suggestions for you that you can consider adding to your code:

  • When u use images as a background u should use background-repeat and background-size

For example, in this case:

    background-repeat: no-repeat;
    background-size: contain;

I hope this helps you

Marked as helpful

2

@thirraz

Posted

firstly, congratulations for complete the challenge!

besides @Renzorr tip, you could use semantic tags for turn your code more legible. Here is a link about it and may help you Semantic HTML

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