Design comparison
Solution retrospective
Proud for completing the project.
Community feedback
- @jasoneczekPosted 8 months ago
I have a few suggestions that I hope you will consider:
Inside of your h2, you'll want to wrap the text in an <a> tag. This is the link where the user would navigate to the blog post.
I don't think you necessarily need to use extra divs for the image and content.
The 'Learning' text can just be a paragraph. You have an h2 below this, so having an h5 there is out of order.
Use rem units instead of pixels for font sizes and sizing.
Your solution looks good but I think with these adjustments it will be more semantically correct. I hope this helps.
Marked as helpful0@ABU-BAKAR-SPosted 8 months ago@jasoneczek thanks for your feedback next time I will consider your advise and apply these approach to create perfect project as it is small or big.
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