Page 1 of 2 12 LastLast
Results 1 to 10 of 11

Thread: Website review

  1. #1
    Join Date
    Nov 2007
    Posts
    346
    Thanks
    0
    Thanked 0 Times in 0 Posts

    Default Website review

    Can anyone please tell me which of these pages look better? http://afesports.com/indexxxxx.php OR http://afesports.com/ind.php

    And any suggestions on making them better? Thanks!

    Edit: Do those shades of blue look dead? How can I make it more lively? Thanks!

  2. #2
    Join Date
    Dec 2007
    Posts
    22
    Thanks
    0
    Thanked 0 Times in 0 Posts

    Default

    I think the first link looks better. And i also see that you have a png problem for IE browsers lower than IE7.

    Here is some Javascript for it. (Place in the head tag)
    Code:
    <script language="JavaScript">
    function correctPNG()
    {
       var arVersion = navigator.appVersion.split("MSIE")
       var version = parseFloat(arVersion[1])
       if ((version >= 5.5) && (document.body.filters)) 
       {
          for(var i=0; i<document.images.length; i++)
          {
             var img = document.images[i]
             var imgName = img.src.toUpperCase()
             if (imgName.substring(imgName.length-3, imgName.length) == "PNG")
             {
                var imgID = (img.id) ? "id='" + img.id + "' " : ""
                var imgClass = (img.className) ? "class='" + img.className + "' " : ""
                var imgTitle = (img.title) ? "title='" + img.title + "' " : "title='" + img.alt + "' "
                var imgStyle = "display:inline-block;" + img.style.cssText 
                if (img.align == "left") imgStyle = "float:left;" + imgStyle
                if (img.align == "right") imgStyle = "float:right;" + imgStyle
                if (img.parentElement.href) imgStyle = "cursor:hand;" + imgStyle
                var strNewHTML = "<span " + imgID + imgClass + imgTitle
                + " style=\"" + "width:" + img.width + "px; height:" + img.height + "px;" + imgStyle + ";"
                + "filter:progid:DXImageTransform.Microsoft.AlphaImageLoader"
                + "(src=\'" + img.src + "\', sizingMethod='scale');\"></span>" 
                img.outerHTML = strNewHTML
                i = i-1
             }
          }
       }    
    }
    window.attachEvent("onload", correctPNG);
    </script>
    The blue colours used in the titles (EG "Statistics") look good, but different colour links on the a:hover event in the main content part. Try a shade of red, but that's me. Good website as well.

  3. #3
    Join Date
    Aug 2005
    Location
    Other Side of My Monitor
    Posts
    3,494
    Thanks
    5
    Thanked 105 Times in 104 Posts
    Blog Entries
    1

    Default

    Agrees that the first looks better. Lots of problems though. IMage optimization is needed here, your main image icon in the top left is loading really slow.. why? Because it is a HUGE image and you are displaying it very very small.

    Resize the image to the size you want, using scaling in photoshop or even paint (to a degree) will keep it crisp but small.

    Clean up the menus, or use only one, there is a lot going on on that page with no direction.

    As I advise for all layouts, find no more than 3 colors and stick with them through out. too many colors, too many scripts and too much content is a disaster for websites.
    {CWoT - Riddle } {Freelance Copywriter} {Learn to Write}
    Follow Me on Twitter: @InkingHubris
    PHP Code:
    $result mysql_query("SELECT finger FROM hand WHERE id=3");
    echo 
    $result

  4. #4
    Join Date
    Nov 2007
    Posts
    346
    Thanks
    0
    Thanked 0 Times in 0 Posts

    Default

    So I should take out some of the scripts and colors? Which ones do you recommend?

  5. #5
    Join Date
    Aug 2005
    Location
    Other Side of My Monitor
    Posts
    3,494
    Thanks
    5
    Thanked 105 Times in 104 Posts
    Blog Entries
    1

    Default

    I would say optimize your logo, use the colors that are in that, and find one menu that will use all of your links, instead of stacking menus on top. That would be a good start.
    {CWoT - Riddle } {Freelance Copywriter} {Learn to Write}
    Follow Me on Twitter: @InkingHubris
    PHP Code:
    $result mysql_query("SELECT finger FROM hand WHERE id=3");
    echo 
    $result

  6. #6
    Join Date
    Nov 2007
    Posts
    346
    Thanks
    0
    Thanked 0 Times in 0 Posts

    Default

    What do you mean by that? "and find one menu that will use all of your links, instead of stacking menus on top." I have pages such as "Track and field" and sub-menus for it such as videos, races and etc..

  7. #7
    Join Date
    Nov 2007
    Posts
    346
    Thanks
    0
    Thanked 0 Times in 0 Posts

    Default

    Also, is it possible to make the height of the left column 100%?

    Thanks!

  8. #8
    Join Date
    May 2006
    Location
    Sydney, Australia - Near the coast.
    Posts
    1,995
    Thanks
    0
    Thanked 8 Times in 7 Posts

    Default

    Please start a new thread for another question.


    Now, get back on topic.
    Peter - alotofstuffhere[dot]com - Email Me - Donate via PayPal - Got spare hardware? Donate 'em to me :) Just send me a PM.
    Currently: enjoying the early holidays :)
    Read before posting: FAQ | What you CAN'T do with JavaScript | Form Rules | Thread Title Naming Guide

  9. #9
    Join Date
    Aug 2005
    Location
    Other Side of My Monitor
    Posts
    3,494
    Thanks
    5
    Thanked 105 Times in 104 Posts
    Blog Entries
    1

    Default

    Quote Originally Posted by afe View Post
    What do you mean by that? "and find one menu that will use all of your links, instead of stacking menus on top." I have pages such as "Track and field" and sub-menus for it such as videos, races and etc..
    Actually it isn't stacked menus, it's double menus. You still show thedefault menu of Stu's there. It is sitting on top of the one you edited.
    {CWoT - Riddle } {Freelance Copywriter} {Learn to Write}
    Follow Me on Twitter: @InkingHubris
    PHP Code:
    $result mysql_query("SELECT finger FROM hand WHERE id=3");
    echo 
    $result

  10. #10
    Join Date
    Nov 2007
    Posts
    346
    Thanks
    0
    Thanked 0 Times in 0 Posts

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
  •