My portfolio for the portfolio project

Code Feedback
1

Hey everyone, I just finish the personal portfolio webpage project :slightly_smiling_face:

1 Like
2

It’s very good looking. Although there is room for improvement

  • Under your ‘TAKE ME DOWN’ button is a large gap - remove that
  • Your projects aren’t centred (align images to the centre for each column)
  • Your form inputs need to be full width on smaller devices
  • You are using four seperate document.ready functions, you only need one - move all animation functions to one and remove the rest
2 Likes
3

Most impressive!
Your code is really clean and easy to read.
For the sake of code hygiene, you should take out commented code that you’ve left there like

<!-- <a class="navbar-brand" href="#">Brand Name</a> -->

and such.
Overall you did very well :slight_smile:

1 Like
4

@michal9909 - Thank you for your feedback, realy appreciate.
About the gap that you mentioned: You mean the gap between the “TAKE ME DOWN” to his white border?
If yes- I lowered the gap and I fixed all the other things that you mentioned, I just have to center the images as you said. thank you.

@Physikeff- Thank you very much! I miss that, I removed what you mentioned. :slightly_smiling_face:

5

Here is a picture of what I mean https://imgur.com/a/aHqQ2

1 Like
7

In this case I have to say that I like this look … :hugs:

8

Hello Everyone!
i just complete my personal portfolio project.
check it out!
here: https://codepen.io/Hooriahic/full/pVaJeW/

9