Design comparison
SolutionDesign
Community feedback
- @rostyslav-nazarenkoPosted almost 2 years ago
Hi! Great solution
A few suggestions
- wrap your code in
main
element or changediv
tomain
alt
attribute should describe an image or its function, plus don't use word image, for this challenge it should beQR code to frontendmentor.io
- you use the default font family, not the ones in the style guide, you can find them at Google Fonts
- use
h1
and change font-size with CSS, skipping heading levels is not a good practice
Marked as helpful1 - wrap your code in
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