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 Component - HTML & CSS

@lkbobadi

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


What are best practices for the use of CSS implemented in this site?

Community feedback

@pradeeps4ini

Posted

Hi, Laura. How are you doing?

I would like to suggest few changed that you can make to make your css better.

  • Use "max-width: 100%" on images. This way your images will always be the size of the parent element.

  • In the "payment plan"section, instead of using <p> element as the parent container. Use a div and store the icon in <img>, price and annual plan in another <div> and change in a <a>.

<div class="payment-plan"> <img src="./music-icon" alt="icon for the music logo"> <div class="plan"> <p>Annula plan</p> <p>$199</p> </div> <a href="">Change</a> </div>
  • Write descriptive alts for the images. They are for accessibility purpose and would help screen readers know about the image and its purpose.

Marked as helpful

0

@lkbobadi

Posted

@pradeeps4ini Thank you so much for the feedback! Will apply this!

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