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

Responsive Sass, Pug and Webpack

Tin.Pham 250

@tin-pham

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


Very appreciate if anyone can give me any review and comment

Community feedback

P
Ken 4,915

@kens-visuals

Posted

Hey @tin-pham 👋🏻

  • In your markup, <section class="container center">...</section> should be <main class="container center">...</main> and this will fix the accessibility issues. Just, don't forget to generate a new repot once you fix the issues.
  • For the music icon, add aria-hidden="true”, because it's for decoration. You can read more about aria-hidden here.
  • Also, I suggest adding transition: all 0.2s; to the button and the links, this will make :hover smoother.

I hope this was helpful 👨🏻‍💻 Other than that, you did a great job, nicely done. Cheers 👾

Marked as helpful

1

Tin.Pham 250

@tin-pham

Posted

@kens-visuals Thank you so much, man.

0
P
Ken 4,915

@kens-visuals

Posted

@tin-pham no problem, man ✌🏻

1

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