Simple Price Grid Component

Community feedback

• 

  T

  Grace•30,990

  @grace-snowPosted over 3 years ago

  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

  @saakarxPosted over 3 years ago

  @grace-snow Thanks for the feedback much appreciated 💖

  0