Hello and welcome to our community! Is this your first visit?
Register
Enjoy an ad free experience by logging in. Not a member yet? Register.
Results 1 to 6 of 6
  1. #1
    New Coder
    Join Date
    Jul 2005
    Location
    Utah, US
    Posts
    77
    Thanks
    0
    Thanked 0 Times in 0 Posts

    Review my portfolio stie

    The design is simple, but valid CSS, and vaild XHTML. (This should make it almost all browsers friendly if i'm not mistaken). Anyways please rate and view. Please note, I have no gfx program other than paint, and this site is really targeted for all people, including people who have slow connections, so the design is simple. Thanks.

    http://phpfreak.wghq.net/

  • #2
    Regular Coder
    Join Date
    Jun 2002
    Location
    Louisville, KY
    Posts
    279
    Thanks
    0
    Thanked 0 Times in 0 Posts
    Very simple indeed. Might as well make it all text...you can convert your title graphic to plain text very easily. That would cut out another 1.6 kb. If you want to keep it, however, at least add an alt attribute for non-visual browsers.

    Honestly, though, you need to spice it up just a little. Fancy graphics aren't necessary; you can do a lot with css and good color combinations.

  • #3
    Senior Coder
    Join Date
    Dec 2004
    Location
    Essex, UK
    Posts
    2,636
    Thanks
    0
    Thanked 0 Times in 0 Posts
    Sorry but I don't like.

    At 800*600 I have a horizontal scrollbar appearing even though the content is no where near to overflowing, and there is a massive gap at the top before the div begins.

    And you're very wrong in thinking that valid code will produce good results in all browsers. IE and Firefox interpret code very differently.

    Your CSS could be cleaned up alot. I would actually do this for the CSS -

    Code:
    body {
    font-family: Verdana;
    font-size: 11px;
    color: #666;
    }
    
    .box {
    border: 1px solid #DDD;
    padding: 15px;
    width: 300px;
    margin: 0 auto;
    }
    
    p.header, p.advheader, p.header:first-letter {
    font-size: 13px;
    font-weight: bold;
    }
    
    p.advheader { font-size: 15px;  }
    p.header:first-letter { color: #B00; }
    
    a {
    text-decoration: none;
    color: #F60;
    }
    
    a:hover {
    color: #FF802B;
    text-decoration: underline;
    }
    EDIT - Attached screenshot of your site to show you how it looks for me.
    Attached Thumbnails Attached Thumbnails Review my portfolio stie-jonphp.jpg  
    Last edited by mark87; 10-14-2005 at 09:56 PM.

  • #4
    Regular Coder GO ILLINI's Avatar
    Join Date
    Jun 2005
    Location
    USA
    Posts
    634
    Thanks
    0
    Thanked 7 Times in 7 Posts
    i dont like it. to plain.
    Why not thank me?

    http://adamsworld.name

  • #5
    Regular Coder
    Join Date
    Sep 2005
    Posts
    146
    Thanks
    0
    Thanked 0 Times in 0 Posts
    i like it, a fair bit..... but, its too far down the page; you have to scroll down when its not needed, make the actualy site up the top. also you need a different background colour

  • #6
    Regular Coder
    Join Date
    Jul 2005
    Location
    Nottingham NH
    Posts
    208
    Thanks
    0
    Thanked 0 Times in 0 Posts
    On my FF 1.0.7 at 1024x768 the page is a good 1/3 of the way down the screen and it is slightly to the right of the screen.

    It is a liquid site, but with it starting so thin, it's be better elastic of fixed. When reducing my monitor to 800x600 I ended up with a really skinny column.

    Using px as a font-size measurement, people using Internet Explorer will not be able to re-size their text. Try 80% to match your size now, or 0.8em. Fiddle as needed.

    I do not like the red hanging caps that lead letter on what are designed to emulate headings (red doesn't match)...

    Use <h1>, <h2>, <h3>, etc and style those instead of <p class="header"> for the purpose of semantics.

    The graphic needs work.

    I do like the simplicity of the site. Nothing comes between the content and the visitor.

    Hope this helps.
    Mike


  •  

    Posting Permissions

    • You may not post new threads
    • You may not post replies
    • You may not post attachments
    • You may not edit your posts
    •