
Please log in to post a comment
Log in with GitHubCommunity feedback
- @arthur-schevey
At first glance, well done! The only thing that is noticeably different is that the line-height wasn't changed for the main text. It also looks like the title needs to be a link as well, it is currently just text.
Don't forget to set fallback fonts for browser accessibility in case a font is unable to be loaded. By comma separating additional fonts, they will serve as fallback in priority order. For example:
font-family: 'Figtree', sans-serif;
Also don't forget to alt text your images.You reused classes a lot which is great! Notably the
info
class, I pretty much had a new class/id for every element which probably over complicates things.Overall great work!
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