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

@rouftarek

Desktop design screenshot for the Order summary component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

Rio Cantre 9,690

@RioCantre

Posted

Hello there! Good job in this challenge. Viewing the solution, I think you should consider the following as well…

  • Import the attribution style in CSS file and remove the style tag
  • Include description with the alt in image tags. Like this line <img class="hero-img" src="./images/illustration-hero.svg" alt="">
  • Add the background image in the body tag. image/pattern-background-desktop.svg with background-size: cover/contain;, background-position: top; and background-repeat: no-repeat;
  • Add the hover state of the main button in .payment with cursor:pointer; and background-color: hsla(245, 78%, 79%, 1);
  • Add cursor: pointer for the links
  • Refactor this part and wrap it with footer tag
<section>
      <div class="attribution">
        Challenge by <a href="https://www.frontendmentor.io?ref=challenge" target="_blank">Frontend Mentor</a>. 
        Coded by <a href="https://github.com/rouftarek/">RoufTarek</a>.
      </div>
    </section>

Into:

      <footer class="attribution">
        Challenge by <a href="https://www.frontendmentor.io?ref=challenge" target="_blank">Frontend Mentor</a>. 
        Coded by <a href="https://github.com/rouftarek/">RoufTarek</a>.
      </footer>

Above all, Well done! Keep it going!

0

@rouftarek

Posted

@RioCantre Thank you again. This is very helpful.

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