@ywsoliman
Posted
Hi Vishal, I hope you're doing great! I am a newbie but I have some notes if you don't mind
- You should use HTML Semantic Elements instead of div as it doesn't have a real meaning
<div class="rec">
should be<main class="rec">
because it's the main content of the page. - From a semantic point of view, using an h3 without an h2 or h1 is not good practice. And therefore,
<h3 class="heading">Improve your front-end skills by building projects</h3>
should be
<h1 class="heading">Improve your front-end skills by building projects</h1>
and change the font-size if you're not happy with it.
Keep up the good work!
Marked as helpful
@happyvisu
Posted
@ywsoliman Thanks for Suggestion. I am also a newbie.