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

Single price grid component

#accessibility

@Shanvie

Desktop design screenshot for the Single price grid component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Hello guys ,i am happy to say that after completing my first periodic test i have summited the projects any suggestion you want to give me they are most welcomed..

Community feedback

Mike Hayden 1,005

@mickyginger

Posted

Hi Akshita, this looks great, well done.

I think you just need to be award of the semantics of your HTML.

You need to have one and only one h1 tag on your page. Looking at the design perhaps that should be Join our community.

I think semantically "$29 per month" is a single element, and probably not a heading, so perhaps your should change your markup to something like:

<p class="subscription__month">
  <span class="subscription__dollar">$29</span> per month
</p>

Hopefully that's helpful! :D

Marked as helpful

0

@Shanvie

Posted

@mickyginger Thanks for the suggestion:)

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