@Islandstone89
Posted
Hi, good job,here is some feedback.
HTML:
-
Hero image and music icon are decorative images, which needs empty alt text:
alt=""
-
"Proceed to payment" would navigate you to another page, so should be a link
CSS:
-
It's best practice to link the fonts in the
<head>
of the HTML, not import it in CSS. -
Font-size must not be in px. Use rem instead.
-
Max-width should also be in rem.
-
Remove the fixed width for
.card-details
Marked as helpful