Skip to content
Submitted over 2 years ago

Single price grid component

Akshita 👩‍💻•340
@Shanvie
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Single price grid component
  • HTML
  • CSS
1newbie

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
    @mickygingerPosted over 2 years ago

    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
    Akshita 👩‍💻•340
    @ShanviePosted about 2 years ago

    @mickyginger Thanks for the suggestion:)

    0

Please log in to post a comment

Log in with GitHub

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