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 using flexbox

LuckyPort 100

@LuckyPort

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


Every comment and suggestion means a lot to me. thank you <3

Community feedback

Vanza Setia 27,795

@vanzasetia

Posted

Hi, LuckyPort!

You should not have visually hidden headings like this.

<h2 class="sr-only">Annual Plan</h2>
<h3 class="sr-only">Annual Plan</h3>

They are not needed. Make the visible text <h2> instead.

Also, for the visually hidden styling, I recommend using the one from The A11Y Project website.

See visually hidden styling — How-to: Hide content - The A11Y Project

Learn more about visually hidden styling — The anatomy of visually-hidden - TPGi

I hope this little suggestion helps you!

Marked as helpful

1

LuckyPort 100

@LuckyPort

Posted

@vanzasetia Noted!!, thank you for the suggestion i will apply this suggestion on another challenges.‎️‍🔥️‍🔥

0
Vanza Setia 27,795

@vanzasetia

Posted

@LuckyPort

Good! 👍

Marked as helpful

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