This is my Portfolio project

Code Feedback
1
2

Hey there!

I had a quick look and I think it looks nice and seems to be on the right track. Here are some comments I have (in no particular order):

  • With the Bootstrap column classes, where specified, column sizing for larger viewports is inherited from the next smallest column size specified. For example, <div class="col-sm-3"></div> means that the column will span 3 columns on xl, lg, md and sm viewport sizes
  • I don’t think col-xs-* works in Bootstrap 4—the smallest size tier is simply col-*, or col if you want auto sizing. In your case, if you decrease the preview window size on CodePen, you will see that col-xs-* never comes into effect
  • The image at the top is currently broken
  • I’m not entirely sure about Comic Sans as a font in a professional portfolio

That’s all from me! Good luck! :slight_smile: