
Design comparison
Community feedback
- P@Theosaurus-RexPosted 3 months ago
Hey @rkrhlikar, great work! Love that you included proper descriptive alt text for the QR code, that tends to get missed by a lot of people even in production code :)
Just one note - heading elements have to be in numbered order from smallest to largest on a page, and the first heading must always be a
<h1>
. If this was being used as a component in a larger page, having a<h2>
makes sense, but since this is on a self-contained page,<h1>
would be the semantically correct choice.Other than that, this looks great!
0
Please log in to post a comment
Log in with GitHubJoin 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