Design comparison
Solution retrospective
I'm quite happy with how close I was able to get to the original design. I really dug into the Figma file to grab as many values as I could (heading size, kerning, border radius, drop shadow offset and blur, etc.).
I didn't focus much on responsiveness this time, but will do so for any future challenges I work on.
What challenges did you encounter, and how did you overcome them?I didn't know why my text kept showing up bolder than it actually was in the browser, so I did some research and fell into a rabbit hole about font smoothing. I was able to figure out why it was happening, though! :)
What specific areas of your project would you like help with?If there's anything I should consider in terms of best practices, I'd love to hear it. Any other feedback is also welcome!
Community feedback
- @LauraGlabPosted 7 months ago
You did an amazing job! I would just change the article tag to a main tag. A main tag wraps all the content in your website, which in this project as you see there's not a lot ;) An article tag is used, when you have more content and you have to seperate it .
<main> <article></article> </main> I'm not saying it's bad, but it's better to use it like that ;) Happy coding!Marked as helpful0@ahanlixuPosted 7 months ago@LauraGlab That's a good point! I kinda just went with <article> since it's often used for cards. But on the other hand, cards usually aren't all by themselves like in this challenge, so it makes sense that <main> might be a better option in this context. Thanks for the feedback!
1
Please log in to post a comment
Log in with GitHub
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