FinzyPHINZY• 80
@FinzyPHINZY
Posted
great work man
Marked as helpful
0
Design comparison
Solution retrospective
What are you most proud of, and what would you do differently next time?
Proud for completing the project.
Community feedback
P
jasoneczek• 320
@jasoneczek
Posted
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 helpful
0
ABU-BAKAR-S• 380
@ABU-BAKAR-S
Posted
@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 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