Welcome to the Treehouse Community
Want to collaborate on code errors? Have bugs you need feedback on? Looking for an extra set of eyes on your latest project? Get support with fellow developers, designers, and programmers of all backgrounds and skill levels here with the Treehouse Community!
Looking to learn something new?
Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and join thousands of Treehouse students and alumni in the community today.Start your free trial
Henrik ChristensenPython Web Development Techdegree Student 38,322 Points
I just finished a portfolio project (practicing) and I would love to get some constructive feedback.
I know the site is very grey (will be updated at some point) and the portfolio project I made in the site is the same in every box because I don't have anything to put up yet :-P
Thank you very much in advance! :-D
Kristopher Van SantCourses Plus Student 18,830 Points
Hey there Henrik! Just took a look at your portfolio and it's coming along very nicely, great job!
A few things I noticed that might help...
I think you could get away with not having your photo over the main image. Since you do have this grey color scheme going on, I think removing the photo and centering and increasing the font-size of "Hello, I'm Henrik" would be more striking and attention grabbing.
I like the idea of having a shadow around your boxes. Maybe instead of having a shadow only on the bottom, have it around the whole box so that they pop even more.
Very nice responsive layout as well!! For certain screen widths I'd recommend maybe aligning the paragraph text in the project boxes to the left. I think in particular between 845px-100px, when these boxes become narrow, a the font could be aligned left, or the boxes themselves could be wider. And at the smallest screen widths 300px-400px the text there could be alined left as well.
I'd also like to see the mouse change when hovering over the facebook and twitter links at the bottom of the page, as well as the send button so that it's clear these are things that can be clicked.
The paragraphs in the About and Skills section are a little hard to read. I think they could use some list-styling or break them up a little more by adding space between each separate thought.
Well these are just some of my personal thoughts! Hope they help or give some food for thought. Very nice job, and awesome to see someone using CodePen! Keep up the great work!