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 - built semantically html

@progchris00

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! Any feedback is welcome!

Community feedback

Bryan Li 3,550

@Zy8712

Posted

Your site looks pretty good. Main thing that stands out to be as being needed to be fixed is the background image. To fix it and make it closer to the original design you need to do two things:

  • remove background-position: 0% -50%;
  • and add background-size: contain;

Hope this helps 👍

0

@progchris00

Posted

@Zy8712 Thanks mate! I had a hard time for the position of the background image that is why I tried using background-position. I will try that! Thank you

0

@BhateshKhairnar

Posted

  1. fix background svg
  2. fix the link hover effect
0

@progchris00

Posted

@BhateshKhairnar Hey mate, thank you so much for your feedback! I understand the number however I am a bit confused on the second one, can you explain it further? Thanks!

0

@BhateshKhairnar

Posted

Hi @progchris00,

Actually, I was trying to say when users hover on link text decoration should be none.

I hope it helps!

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