Hello!
good job in posting your first solution :)
What I noticed:
- you correctly implemented the border radius for the card, but a border radius is missing on your image (see original design provided)
- in order to use semantic html, you could use <main> instead of a simple <div> for the card
- for your text, consider using headings such as <h1> for your main title, so that you are structuring your content effectively.
The last 2 comments would not optimised how your page looks but would be helpful for SEO and accessibility purposes.
Hope this helps!
Marked as helpful
@shubhamr10
Posted
Hi @margaux-works As per your suggestion I tried to improve by using semantic tags. Do check the solution. Thanks
The width might be slightly different.
You can see my solution here: https://www.frontendmentor.io/solutions/responsive-blog-preview-card-YCK-HD2HgF