Personal Portfolio- Tejas

Code Feedback
1

http://tejas-freecodecamp-portfolio.s3-website-us-east-1.amazonaws.com/

1 Like
2

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 <h1> lives :slight_smile:
  • The div with the class of “card_image” could become <figure class="card_image"> because this div stores only a picture :slight_smile:
2 Likes
3

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".