Peer-to-peer code review.

HectorTorresE / personal-portfolio

0 stars 0 forks source link

Peer-to-peer code review. #10

Open fed1k opened 2 years ago

fed1k commented 2 years ago

Good Points

The whole webpage looks awesome

Suggestions :
- Giving z-index: 2; for navbar so that it appears instead of going behind some sections
- For welcome section, button should be fixed because when it is clicked it stays on click position even if you move your mouse and also change its color because when you click it text color is not appearing.
- Add a to your icons that should direct user to that website

jpol01 commented 2 years ago

Great job!

Kakalanp commented 2 years ago

Your project looks great! 🌟 Good job! 👍

After going forwards and backward, I noticed just one bug, the one pointed out by @fed1k, your navbar is behind some images: You'll only need to change the z-index to fix this and that's it. 🎉

Good luck! 🍀