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! While you're at it, check out some resources Treehouse students have shared here.

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

General Discussion

Feedback on my portfolio please

http://www.dan-croft.com My portfolio is just about finished I was hoping you guys on the forum would take a fresh look at it please tell me your likes/dont likes etc. I am considering changing the About me Section. Was going to do progression bars etc but from Nick Pettit 's live cast he seems to be against them so i have gone for the "Ticks" but i think this section of the site looks unclear.

Also if anyone spots any bugs in different resolutions or browsers please let me know i have checked it as much as i can and dont see anything obvious.



3 Answers

James Barnett
James Barnett
39,199 Points

It seems like your professional projects have a lot more polish than your own portfolio. I like the layout of the headers, the recent projects & the social icons.

In particular the contact form and the text in the about section could both use some work.

I think it could use some work on the UX, hierarchy, typography, transitions and over all look and feel.

I was just looking now for some info on them. Thank you for the links will take on what you have said on board and make changes where needed. Also what transitions need working on?

James Barnett
James Barnett
39,199 Points

I like the design/concept of the transitions you are using with your screenshots & social icons but I think the actual transition and timing could use some work.

In the big picture, you have to make a portfolio with an audience in mind. Are you trying to sell to clients? Are you trying to sell to other freelancers? It's unclear from reading your site for me who exactly you're selling to. You also claim "user interface designer" but then have no UI work examples on there.

Portfolio sites are massively overrated, IMO. I think case studies are far better when selling to clients. I can look at the pretty catalog all day but how does that benefit my business? Take one of the sites you did, explain what you improved and what benefit the clients received. Also, if it's client facing, I don't care about your skillset. You should have this on here for other professionals but it's fairly meaningless to clients. Might be a reason to reconsider a single-page site.

The overall design is a pretty good idea but the typography needs work. This page cries out for a max-width. Your line length is unreadably long and your paragraphs either need to be indented or spaced out.

You also have two bad breakpoints where the header links are overlaid on your hero image and where the contact form at the bottom is messed up. It's a pretty narrow width, so check your numbers on that.

Keith Doyle
Keith Doyle
25,973 Points

I really like the main image up top. Nice combo of wood/metal (high tech/low tech).

One thing I noticed is that when I click on a link in your main navigation and I'm taken to that section, the top of the section is cut off ever so slightly.

And this is super minor, but it seems that the Twitter icon isn't centered in your oval like the rest of the social icons, or maybe I just need an updated prescription.

Update: Played around a little more and your hire button isn't fully clickable, only the text. Make sure you double check your copy for grammar, punctuation and so on. There are a few typos such as "Geocitys" should be spelled "GeoCities". To make it easy, copy and paste it into a program like Word to get possible typos since not all text editors correct grammar. Make sure you break up paragraphs using separate <p> tags instead of using <br> within a <p> tag.