@FluffyKas
Posted
Heyo,
Your solution looks really great! Only have some small suggestions here:
-
Background patterns should be added with css.
-
You could perhaps wrap the attribution in a <footer>, that would get rid of the issue in the accessibility report.
-
I'd not use the same heading for the Annual Plan text and for the price. The price itself could just be a paragraph and the Annual Plan an <h2> if you follow heading levels in the correct order.
-
You could remove the alt texts from the images. They're arguably decorative only (the music icon is, for sure) so their alts can be left empty.
Marked as helpful
@snowsneakers
Posted
@FluffyKas Thank you for the feedback!