It looks good!
Currently, the projects are not clickable — but you are most probably aware of that.
I’d consider changing:
<title> Portfolio </title>to something more descriptive like 'Tejas Sabunkar Portfolio"
- Actually, you don’t need the nav to be inside a header — consider taking it out to make page navigation easier;
- Your section with id “welcome-section” should be a
<header>because that’s where
- The div with the class of “card_image” could become
<figure class="card_image">because this div stores only a picture
Yes, I am aware about those projects are not clickable.
Sure I will be considering your detailed feedback.
“Words cannot express how grateful I am for your help and such wonderful feedback".