
Design comparison
Community feedback
- @arthur-scheveyPosted 2 months ago
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!
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