SitePoint Sponsor

User Tag List

Results 1 to 12 of 12
  1. #1
    SitePoint Wizard masm50's Avatar
    Join Date
    Oct 2001
    Location
    UK
    Posts
    2,508
    Mentioned
    2 Post(s)
    Tagged
    0 Thread(s)

    Please critique my site

    Hi everyone,

    I would really appreciate it if some people could take a look at http://www.audiomelody.com and give me some critiques.

    Be as harsh as you think necessary

    Tim

  2. #2
    SitePoint Zealot HeatherID's Avatar
    Join Date
    Jun 2003
    Location
    Maine, USA
    Posts
    136
    Mentioned
    0 Post(s)
    Tagged
    0 Thread(s)
    I like the color scheme, the red really pops. I'd made your Google ads a different color, if at all possible, because your main focus should be your content, not your ads. The layout looks nice too, pretty standard three-column. You've also included the ads to be fairly non-instrusive, which is a benefit to most users. Overall, looks pretty good!

    -Heather

  3. #3
    SitePoint Member
    Join Date
    May 2004
    Location
    ireland
    Posts
    6
    Mentioned
    0 Post(s)
    Tagged
    0 Thread(s)
    looks ok, maybe a non white background though, looks kinda like something is missing

  4. #4
    SitePoint Wizard jag5311's Avatar
    Join Date
    Jan 2003
    Location
    Somewhere in Indiana
    Posts
    3,082
    Mentioned
    0 Post(s)
    Tagged
    0 Thread(s)
    Be as harsh as you think necessary
    ALL RIGHT!!! Lets begin j/k.

    A few points to note.

    1. This is not too bad.
    2. I think on the left, for your menu, you could probably remove the left padding, there is just too much red space between the left side and the words.

    btw - I bet you are using Dreamweaver MX with the XHTML Compliant box CHECKED when you go to FILE NEW, aren't ya

    Nothing wrong with that, unless you are NOT going to try and make an XHTML compliant website.

    This

    Code:
     <?xml version="1.0" encoding="iso-8859-1"?>
    
    <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"
        "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
    Is at the top of your webpage in the source code. Your site is not even remotely in the same
    area code as a standard compliant website, because of your nested tables, width, height, border, color attributes
    you have assigned many things. I do have to commend you for not using the dreaded FONT tag though.

    Also, in your content, make the MAIN link something that stands out. I got lucky to know it was a link because
    my mouse hovered over it and I saw an underline. Make it a different color or something.

    Thats more of a usability issue.

    Oh, add a TITLE. <title>Whatever</title> in between your <head></head> tags.

    You are using unordered lists for your menu's which is a good thing, but why don't you style them up a bit via
    CSS.

    These are just a few points I wanted to make. Hope they help.

    Bryan

  5. #5
    SitePoint Wizard masm50's Avatar
    Join Date
    Oct 2001
    Location
    UK
    Posts
    2,508
    Mentioned
    2 Post(s)
    Tagged
    0 Thread(s)

    Thanks for the replies

    I really appreciate all the commments guys - thanks

    I'm only using Dreamweaver 4 at the moment (not MX), but what do you think I should put as the doctype?

    Making the Home link (is that the one you meant - in the left nav?) more noticeable is very useful - I'll work on that. Just like the title tags

    One more question though - whats an ordered list as opposed to and unordered one - I'm a bit lost on that one?

    Thanks again,

    Tim

  6. #6
    SitePoint Wizard jag5311's Avatar
    Join Date
    Jan 2003
    Location
    Somewhere in Indiana
    Posts
    3,082
    Mentioned
    0 Post(s)
    Tagged
    0 Thread(s)
    Ordered list

    <ol>
    <li>Item 1</li>
    <li>Item 2</li>
    </ol>

    That will output

    1. Item 1
    2. Item 2

    Unordered list is

    <ul>
    <li>samething</li>
    <li>Samething</li>
    </ul>

    that just gives you bulletpoints.

    For the Doctype, I would use,

    <!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN" "http://www.w3.org/TR/html4/loose.dtd">

    That allows you to continue to use all the attributes like width, border, margin, etc...

    Under XHTML, there are stricter rules involved that won't validate if you use what you do in your site. For example, look at the source code of my site, Juiced Thoughts. Its all either divs, id's, or class's. I am separating the presentation from the content. THe presentation is all in the borders, widths, margins, etc...

  7. #7
    SitePoint Member
    Join Date
    May 2004
    Location
    MA
    Posts
    3
    Mentioned
    0 Post(s)
    Tagged
    0 Thread(s)
    Definitly remove some padding

  8. #8
    SitePoint Member theSpear's Avatar
    Join Date
    Jan 2004
    Location
    Georgia
    Posts
    20
    Mentioned
    0 Post(s)
    Tagged
    0 Thread(s)
    eww popup that passes google popup blocker...
    could use a descriptive <title> tag in your <head>
    maybe a small border for the content of the site.. 1px grey..

  9. #9
    SitePoint Zealot DanMan108's Avatar
    Join Date
    Nov 2003
    Location
    Geek Valley
    Posts
    137
    Mentioned
    0 Post(s)
    Tagged
    0 Thread(s)
    Aesthetically, I like it. I'm not a huge fan of the advertisements, especially the pop-ups.

    However, great site. Is it missing a <title> ?

    Make sure you run it through http://validator.w3.org.

  10. #10
    SitePoint Member Guldstrand's Avatar
    Join Date
    May 2004
    Location
    SWEDEN
    Posts
    24
    Mentioned
    0 Post(s)
    Tagged
    0 Thread(s)
    Hmm!? What can i say...

    First of all i think you need more pictures on your site, (it lookes a little boring with all the text and no images)

    Second: The right side (under "Log In" section") lookes a little emty.

    Otherwise itīs easy to navigate, and the sections are great.
    "Itīs not stupid to ask many questions,
    the stupid thing is NOT to ask them"

  11. #11
    SitePoint Enthusiast
    Join Date
    Aug 2003
    Location
    PA, USA
    Posts
    35
    Mentioned
    0 Post(s)
    Tagged
    0 Thread(s)
    Hi masm,

    I am going to have a bash at reviewing your site. I tried to review it as a user rather than examining the HTML too closely, since most of the other comments covered that pretty well;

    1. The descriptions in the Directory listings were somewhat sparse looking.

    2. try to avoid using a pop up window for site links.

    3. I like the logo on the right hand side, but with the banner, the logo and
    the animated (and _very_ annoying) banner advert, it makes the fold look
    quite disjointed.

    4. On your pages which rate and classify software (i.e. 'Audio Editors') you
    might be better to use a rating system otherwise it looks a bit sparse. Maybe
    also have some statistics about # topics in each category?

    5. I liked the colours, very easy on the eye.

    6. Maybe you could try this for the google adsense portal box and see if it
    gives the site some better overall consistency; Change the google header 'Ads by Google'
    background to be the same as the red body background and then place the entire add inside
    one of your site portal boxes and call it Ads or something? Not sure if it would look
    better, but it might be worth a try for consistencys sake

    7. Break your articles into 5-10 per page and put a navigation bar in place so that you
    can page through them.

    Hope they weren't too nitpicky!

    (P.S. I noticed you're from Milngavie; I'm from Helensburgh!)

  12. #12
    SitePoint Wizard masm50's Avatar
    Join Date
    Oct 2001
    Location
    UK
    Posts
    2,508
    Mentioned
    2 Post(s)
    Tagged
    0 Thread(s)

    Thanks for the help (again)

    Thanks for the help and the answers to my questions guys

    I'll make the changes you guys have suggested over the weekend.

    PS. - maaxiim - Always good to find someone so close

    Tim


Bookmarks

Posting Permissions

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