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

Simple Price Grid Component

SaakarX 320

@saakarx

Desktop design screenshot for the Single price grid component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

T
Grace 29,310

@grace-snow

Posted

Hi,

This looks pretty good. There's just an issue with html headings. It's really important that headings go in order and don't jump levels. So a h3 belongs to a h2, a h4 belongs to h3 etc.

Also, headings must make sense as stand alone headings in a document outline, a d text cannot only be in a span/div.

So this should be changed from

<div class="price">
    <h2>$29</h2>
    <span class="pm">per month</span>
</div>

To this

<div class="price">
    <p>
        <span class="price-highlight" >$29</span>
        <span class="pm">per month</span>
    </p>
</div>
1

SaakarX 320

@saakarx

Posted

@grace-snow Thanks for the feedback much appreciated 💖

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