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

Used Flexbox

ArslanJ9 150

@Arslanj9

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


Anything to make it better would be appreciated!

Community feedback

@fabricioceppi

Posted

Hello Arslan. Good Job! It's looking pretty good. Here are some suggestions to solve those accessibility problems: Change your <div class='bg'>...</div> to <main class='bg'>...</main>, the h3 for an h2 and the h5 for an h3, and lastly add an alt attribute to your <img src="images/icon-music.svg">. In this case the alt text can be 'decorative', but is necessary for a correct markup.

Have a good day!

Marked as helpful

1

ArslanJ9 150

@Arslanj9

Posted

@fabricioceppi Thank you so much for your valuable feedback. I was just wondering how can it makes the page better by giving all these changings because it won't give any changings to the outcome! Your answer would be appreciated. Thanks again!

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