Treehouse: Grow your CSS skills. Land your dream job.

Site review please

  • # March 15, 2012 at 5:18 pm


    Please review my site and tell me what you think …


    # March 15, 2012 at 10:45 pm

    Hey James. If it’s just going to be that text, it’s gotta be way more narrow, maybe something around 600px. Reading across very wide areas is uncomfortable (which is why you see so many columns in magazines/newspapers).

    # March 16, 2012 at 1:14 am

    My recommendation is that since it’s only text you work on some creative ways for people to interact with the site to see the different sections. You can achieve an elegant look and feel with a minimalist approach, but you’ve got to get away from what people call a “wall of text”.

    I wish I could think of some really good examples, but it’s escaping me. Maybe someone can remember a couple sites that focus more on typography? I believe Chris Burton has a particular niche in that arena.

    # March 30, 2012 at 6:08 am

    Thanks @JoshWhite

    Hey, @JamesBarnsley. I love the typeface you have chosen for the site. I do think they have an excellent point. You have way too much text going on here, man.

    Your projects section will not work. Unfortunately, no employer will sit through those paragraphs to read about the project. They would much rather see what you’ve done visually.

    The section right before skills, seems very odd to me. Some of the description could be better distributed. I would also add more line-height to the body copy.

    On a side note – If you’ve bought a domain, why not use a more personal email like

    # March 30, 2012 at 7:01 am

    I agree with the above, way too much text, not enough visuals. If you do want to display lots of text, columns are the best route to go as it splits content and gives the reader an opportunity to take in the content in bitesize amounts.

    # March 30, 2012 at 7:49 am

    Yes, too much text. I would look at more creative, fun ways to display the text. Also, I would avoid a completely black font, because it´s too in your face for my tastes.

    Type in ´portfolio inspiration´ into google, and see what takes your fancy.

    # April 1, 2012 at 10:42 am

    add to your #page

    max-width: 800px;
    margin: 0 auto;

    and add additional left/right padding to your

    tags of 24px

    Also, try a different color on your headings (like #A69572)

    # April 1, 2012 at 10:51 am

    @jamesbarnsley – looks like you have worked on some interesting stuff. Maybe you should include some screen shots to give the site some visual appeal. It would also be good if you were able to demonstrate your abilities with dynamic code by using it in your site.

Viewing 8 posts - 1 through 8 (of 8 total)

You must be logged in to reply to this topic.