I’ve finally reached the end of the first unit! I’m pretty satisfied with the progress I’ve made between my first project and this one, and now I have to resist the urge to go back and tweak them. Anyway, I’d love some feedback on my final project, the portfolio website. Here’s the link:
Thanks in advance!
2 Likes
Good job! Keep at it. I love that you made it responsive as well. 
1 Like
I do my best to make everything responsive! It’s a little more work but it’s nice extra practice.
Indeed it is! I hope you have fun and learn a lot with JavaScript!
1 Like
Don’t resist the temptation. Each one of the projects builds on the previous one. A concept that you learned in the lessons is built on in the first project and then that is built on in the next and you’re being introduced to another concept, etc.
No web app/web page remains static after it’s put out there. The MVP (Minimum Viable Product) is put out and then it is built on and improved.
Think of Windows putting out a beta and then the final product. What’s the difference when you put out a page and then go back and improve on that page as your experience grows?
1 Like
That’s true! I feel like I should at least wait until I’ve learned some JavaScript and then I’m definitely gonna double back and indulge on at least my portfolio page. Probably implement some ideas I had for the others that weren’t possible with CSS alone.
Yes on using things from JS to update/improve your portfolio.
1 Like
I like the colors and the use of the color gradient.
My two cents…
- Put your name in the navbar, say, on the far left. Like it or not, you’re branding yourself.
- Pad those navbar links.
Keep it up!
2 Likes
Was thinking of doing that, this pushed me over the edge and I think I’ll go back and do it tomorrow when I get back on.
1 Like
excellent use of gradient, it looks very nice. Besides the lack of padding in your nav bar you could work on the contact me section, the inputs could be aligned a little better. Your nav bar also has an a tag as a child of your ul tag. Its best practice to have your a tag be a child of the li tag. so it should be ul>li>a instead of ul>a>li. also your for attributes in your labels are missing corresponding inputs with the same id
good luck 
1 Like
I was wondering about the ul > li > a thing. The correct order made more sense to me but when I looked at some other code for reference it was the other way. Guess I should have trusted my instincts! When you say the inputs should be aligned better do you mean they should be aligned on the left and the right? Or is there a problem with the current left alignment I’m not seeing? I wasn’t sure if they should be the same length and the email bar always looks a little long to me. Thanks for all the feedback I’ll be sure to start fixing things here in a few!
Nice!
I like the color combination you used. 
1 Like
by the alignment i meant that they should be the same length, it would also be useful to set your textarea’s resize property to ‘vertical’
1 Like
I love the color gradient as well ! Also nice it’s responsive
1 Like