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