Dharmik_487• 1,740
@Dharmik48
Posted
Hey👋,
Great job with the solution! Looks nice. I would just suggest you to add some transition to hover states.
Keep it up👍
Marked as helpful
2
Design comparison
Solution retrospective
please share your thoughts and tell me what should i improve ?
Community feedback
ogolajecinta• 45
@ogolajecinta
Posted
Hi Abdallah, I've gone through your challenge, and most of the things are pretty much okay. Good job. Except for a few; apply font-weight to the elements as per the styles.md file. Within the article tag, enclose the content with a paragraph tag, then set a line-height for it for spacing between the lines. There's a typo on the button tag. Lastly, the button does not require an anchor tag. It works well without. That's all.
Marked as helpful
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