@CecDorWEB
Posted
Hello, Congratulations on your solution !
Things you could improve ✍️:
- Use the font indicated in the style guide with the link to go to the web site. It's possible to use <link> or @import to retrieve the font.
- Try using semantic HTML elements like main, section and article.
- I suggest using clear descriptive CSS classes like .card, .card-title and .card-description.
Happy coding! 😎
Marked as helpful
@clarkjr2016
Posted
@CecDorWEB I've updated this project with your suggestions. This was very insightful feedback and I appreciate you taking the time to review my code.
You're suggestions on the semantic HTML was spot on. I missed that on the first go but it was fairly easy to correct after reviewing!