
Please log in to post a comment
Log in with GitHubCommunity feedback
- P@markobrien7
The solution looks excellent to me. Liked the way you have imported the google fonts, haven't saw it done like that before.
You are using semantic html and have provided an alt text on the image for accessability. The only thing I could possibly suggest is have the main part in a <main> tag since you also have a <footer> but having it as a <div> is perfectly fine.
Marked as helpful
Join 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