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

Design

Amy McKnight
Amy McKnight
10,124 Points

Would you give me some feedback/ second opinion on my portfolio site?

I just finished my portfolio site http://amymcknight.com/ I built after going through the How to Make a Website / Basic CSS classes.

This is the first time that I've EVER built an entire site from scratch using just HTML and CSS. (I've been using the artisteer program in html/css mode) and before that I build sites using the Joomla CMS.

I'm taking another coding course. . . some place else and the teaching assistant is a designer. I asked for feedback and she basically said that it looks kinda dated and I wouldn't be able to use it to really get a decent paying job or decent paying freelance work.

I sincerely would like a second opinion. I feel like she was a little harsh. But if you think she was right in her assessment. I'd rather know now than later!

Thanks in advance!

Amy

http://amymcknight.com/

4 Answers

Flavio Carvalho
Flavio Carvalho
26,636 Points

Hi Amy,

It's not terrible, and here are just a few tips/suggestions (I mean it all as constructive criticism ;) ,don't take anything personal, and if it seems like I'm being an a** I apologize):

  • I like the fact you chose different fonts other than the usual "Helvetica", "Open Sans", gives your work a bit more personality. I'd suggest a little work on the line-height of <p> tags and a little kerning, so it's more pleasant to read (basically, make the words breath a bit). For that I'd recommend Treehouses course on Typography:

http://teamtreehouse.com/library/web-typography

It's by FAAAR and beyond one of THE best courses I've done here on Treehouse and to this day I come back to it for references. (no, they are not paying me to say this). It will give you some excellent tips on working typography, text, paragraphs and titles.

That was one major thing I wanted to point out.

A second thing is (maybe!*) review your color palette choices. You have some contrast going on, which is great, but some colors are just too bright (some greens and reds), and using them as background colors or in logos is a little much for the eye, and look a bit off... in other words "not designer work material". You ARE in the right path, just review the brightness or other color combinations ;) Look up some inspirations on sites like Bechance, or Dribble; see colors that they use and play around. Color schemes are a tricky one to get around, but with some time and researching (looking other's works) you start to develop a better eye for it. That being said, I like the purples you chose for your home page.

Small detail that makes a big difference for whoever comes to look at your website: This pictures are too small when clicked:

The header you have on the front page is not the same as the about or contact pages, with you logo and "Front End Web Developer"... make them consistent, it calls for more attention to what you do.

That's my 2 cents :D hope if nothing else at least gives you some food for thought. Good luck

LaVaughn Haynes
LaVaughn Haynes
12,397 Points

I'm going to have to bookmark that typography course! Thanks

Flavio Carvalho
Flavio Carvalho
26,636 Points

Do it without looking back! Worth every pixel.

Amy McKnight
Amy McKnight
10,124 Points

Flavio,

Thank you so much for the constructive feedback! That is what I wanted!. I will definitely add the Typography course to my list to do.

And I will check out some of the designer resources. Color scheming is something that I need to work on. But the fact that you liked the purple combo is encouraging that site, is the first thing that I did completely on my own. So maybe there's hope for me. I usually use what artisteer recommends or some other (obviously not good) inspiration :-)

I'm thinking that I just need to take out all the sites that are not live anymore/ I don't have a way to store a live copy on my own server. Some of the sites are live. A few were hacked a couple of months ago and I don't know how to fix them. They were done with an older version of Joomla. And I don't know enough to connect the database with a new front end. One of the reasons I decided to learn the real nuts and bolts of HTML and CSS so that would not happen to me again.

I was going for the "I have experince" look like LaVaughn mentioned. But less may be more. The site that are mine and I can redo, I'm going to make them better and then just put them up instead of the ones I only have a (teeny tiny) picture for.

Finally I think you are right about the consistency in the Header. That big block of text isn't doing my any favors so I'm going to just delete it.

Thank you again for the second opinion. I'm definitely going to take your advice!

All the best,

Amy

This is surely just my opinion, but I would say she's pretty much right about what she said. The whole combination indeed looks like outdated and therefore it's hard to be on top of competition with current web design market.

What I would do differently? Well, I would change the colors, make some light borders around each portfolio item, change the line heights for text, maybe make an full width footer with contact information and so on. Also I would make sure that the home page is the first navigation item as well. For SEO purposes, every page should have different tittle tag and you should write an meta description for every page as well.

One good idea might be to put all information from these three pages to just one page? Your landing view could be a full size picture with some partly transparent color layer on top of it and some short decription in the middle and navigation on top. Scrolling down shows what services you offer, your portfolio, more about you, and finally contact information with contact form.

There's so many options, and there was just some of them. Anyway, that's still pretty good work for a first handmade site design. Good option is to look at how others are doing same things, but don't just copy someones page, get some ideas and make them "your way".

edit... couple minutes too late after Flavio, partly with same suggestions :)

Amy McKnight
Amy McKnight
10,124 Points

Hi Riku,

Thanks for the feedback. I'm going to do the thing with the titles and meta info. I thought of doing a one page thing but I haven't gotten finished the JavaScript thing and don't know how to do a scrolling menu. As yet. I thought of just using a template. But since this is my "Portfolio" I wanted to actually code it and show what I actually know how to do.

But I do think you have a point. One page sites are the new black. When my skills get better I may go that route.

Thank for the honest feedback!

All the best,

Amy

Actually you only need javascript for one page site, if you want scrolling to be smooth. With css you can use fixed positioning for navigation bar, so it will stay on top all the time.

Also if you will keep the text content in your heading, put some left and right paddings there, so your text won't get too close to edges on some devices.

Happy coding!

Amy McKnight
Amy McKnight
10,124 Points

Thanks! that's good to know, I'll look into it. I'm so new to all this and want to learn it all NOW. But knowing that there is one more thing I can use what I understand (a little) CSS, more is great!

Thanks!

LaVaughn Haynes
LaVaughn Haynes
12,397 Points

First of all, congrats! Personally, I think that your site looks good. I do think that some of the projects look a little dated. On the other hand it shows that you've done a lot of projects. I can't speak to whether it would hurt your prospects to find work. I've never been involved in any hiring processes and I'm not a designer.

I heard a developer on one of the podcasts that I listen to say that although he's NOT currently looking to leave his job, he applies for jobs every once in a while to stay sharp, test the market, and see what skills people are looking for. You could always apply for a few jobs and register with agencies in your area to see what kind of response you get. That will certainly give you a good assessment of how other professionals view your work.

I would say stick with the courses and maybe come up with some personal projects to practice what you learn. Add those to your portfolio. As they get better swap out some of your older projects. Maybe you'll get more freelance work and you can also apply your new skills to those projects as well, and add them to the portfolio.

Other than that, I would say (if possible) link to some bigger thumbnails or incorporate a modal window. The current images are too small http://amymcknight.com/img/medium_mynhcg.jpg

Amy McKnight
Amy McKnight
10,124 Points

thanks so much LaVaughn!

I realize that in asking for feedback on my portfolio I was . . . asking for feedback on my portfolio. All the work that I've done up to this point! I actually was wanting to know how people like my actually portfolio site. But this has been great! I'm glad to be able to get all of this great feedback. It will help. But thank you so much for letting me know that my first go in to the no WYSIWYG world wasn't a total flop :-P

That is a good idea about applying for jobs. Maybe things I'm not too pressed if they say know and see what type of feedback I get. Hadn't thought of that. Good idea.

I'm definitely going to stick with Tree House. I can say hands down this is the best kept secrete on the net. I'm really grateful that I found it. I'm on it learning something new everyday.

I'm going to look in the library for a course on Modal windows. If you know of one, that you can recommend would be appreciated!

Thanks again for the feedback!

Amy

LaVaughn Haynes
LaVaughn Haynes
12,397 Points

It's definitely not a flop. I like it.

Take a look at the course trailer and add it to your queue if it looks interesting to you. Modals are touched on in this course. http://teamtreehouse.com/library/framework-basics

Also, you can watch it now if you want but you might want to take some of the intermediate CSS courses before actually tackling something like this. This 8 minute workshop demonstrates how to do a modal in pure CSS. https://teamtreehouse.com/library/create-a-modal-window-with-css

From a usability perspective it would just be nicer (if you can get some larger images) to not have the user navigate away from your portfolio site unless you are linking them to a live site in a new window.

Add some left/right padding to the purple "I am Amy McKnight" bar so the text doesn't touch the edges when the browser is narrow.

The company I work for has a dated site. I can't use anything that I learn here at work. I applied for some jobs a couple of years ago and the recruiters said, "hey, your stuff looks dated." I passed every code test I ever took. I literally have glowing letters from my previous supervisors. That portfolio carries a lot of weight though. I hopped on Treehouse. Best decision I could have made. It was slow at first because many months I didn't have the time or the extra money to take courses, but I've leveled up my skills over the past year and over the next 8 months I'm working on building a portfolio from scratch. I feel more confident and more proud so I think it's worth the extra time and effort.

Good luck with your portfolio!!

Amy McKnight
Amy McKnight
10,124 Points

Thank you so much! Looked over both of those and have book marked them to watch later.

Thanks for sharing your story, too. It is encouraging.

All the very best in what you are doing!

Amy