Jump to content


Photo

Time for another review...


  • This topic is locked This topic is locked
22 replies to this topic

#1 underparnv

underparnv
  • Members
  • PipPipPip
  • Advanced Member
  • 30 posts
  • LocationReno, Nevada

Posted 03 August 2006 - 10:37 PM

Alright...I finally have my site ready to go for the 2006-07 NFL Season.  All functions of the site were written / designed by myself.

What I would like to know is what everyone thinks, and what changes as well as additions they would make.

http://tse.matrixcs.biz

Thanks!!
"Programming today is a race between software engineers striving to build bigger and better idiot-proof programs, and the Universe trying to produce bigger and better idiots. So far, the Universe is winning."

The Sporting Edge - Free NFL Football Pool


#2 moberemk

moberemk
  • Members
  • PipPipPip
  • Advanced Member
  • 695 posts

Posted 04 August 2006 - 12:05 AM

Your top header is all white for a while as the page loads slowwwwly. Maybe you should shrink it down somewhat. Now overall, I think that this is a nice, clean, CSS design that you've got here, though the columns could probably use a bit more padding, and the green block headers are kind of plain. Maybe a football or something could go there to give it some detail other then a block of colour?
Aside from that, it's good, clean, and reflective of your goal. I can immediately tell what your site is about, and though it is somewhat narrow on 1280x1024, it still looks good. No compatibility errors that I can see, which is good-overall, it's well done. But, just a question-why don't you increase the main column to a full 800px width? If you applied your background image to the body tag instead, it would work just as well.
Disclaimer: No matter how harsh I may be, I'm always IMO only.

#3 cmgmyr

cmgmyr
  • Members
  • PipPipPip
  • Advanced Member
  • 1,278 posts
  • LocationUSA

Posted 04 August 2006 - 12:38 AM

Loading was also a little slow for me.

I like it though.

-Chris

#4 448191

448191
  • Staff Alumni
  • Advanced Member
  • 3,545 posts
  • LocationNetherlands

Posted 04 August 2006 - 06:46 AM

Consider not loading the whole NFL schedule in one page, but per week or better yet use a calendar. It loads very slow and the list is just too long for comfort.

#5 steviewdr

steviewdr
  • Moderators
  • Advanced Member
  • 1,364 posts
  • LocationIreland

Posted 04 August 2006 - 08:12 AM

moberemk is exactly right.

Put an image of a football on the left or somewhere.
Also - I would put the text in the top into an image - this will let you pick a different font and style it up a bit.

The column on the left is way too wide - and the middle one is too narrow.

Looks ok tho.

-steve

#6 ober

ober
  • Staff Alumni
  • Advanced Member
  • 5,337 posts
  • LocationEast Coast, USA

Posted 04 August 2006 - 12:42 PM

I'm not quite sure what the hell you're loading, but I watched at least 1.2mb fly by on the initial page load.  The header image didn't show up until after that was done, and even then it wasn't all that impressive.

Ok, so I looked at your source and got the size of your header image.... 900KB!!!!!!  Are you out of your mind?  That image shouldn't be anymore than 100kb tops!  Do some optomization!

The rest of the site looks pretty good... my only other complaint would be that you should probably think about putting a line under the horizontal nav at the top... it kind of rolls over into the 'user login' and 'latest headlines' areas.  It looks a little weird.

Info: PHP Manual


#7 underparnv

underparnv
  • Members
  • PipPipPip
  • Advanced Member
  • 30 posts
  • LocationReno, Nevada

Posted 07 August 2006 - 04:30 PM

Alright...I got that nasty unoptimized png out of there and put in a streamlines jpg.  :)

I will see what I can do about seperating the nav from the content.  I never would have caught that!  :P

I will also see what I can do to make the schedule page a bit more user friendly, as the way it displays now does kind of stink. 

Finally...I kind of like the plain roll-overs at the top, but again will see what I can come up with to spice it up a bit.

Thanks a ton for all the feedback!!
"Programming today is a race between software engineers striving to build bigger and better idiot-proof programs, and the Universe trying to produce bigger and better idiots. So far, the Universe is winning."

The Sporting Edge - Free NFL Football Pool


#8 ober

ober
  • Staff Alumni
  • Advanced Member
  • 5,337 posts
  • LocationEast Coast, USA

Posted 07 August 2006 - 04:46 PM

Your site looked interesting, so I signed up.  When I got the confirmation email this is the link it gave me:
http://www.http.com/...s.biz/index.php 

You have too many http's in there.  Might want to fix that.

EDIT: actually, that was after it confirmed me.  It forwarded me to that.  The link in the email is OK.

Also, you may want to put on the form what your limitations are.  I put in a 4 character username and had to redo that part of the form. 

EDIT2: it won't do the confirmation based off of the link I got in the email!!

Info: PHP Manual


#9 underparnv

underparnv
  • Members
  • PipPipPip
  • Advanced Member
  • 30 posts
  • LocationReno, Nevada

Posted 07 August 2006 - 06:04 PM

Hey ober,

Sorry for the email mis-hap.  I am working on it now and went ahead and manually approved your account -- but I see you already logged in.

I will get that fixed asap, and will add the requirements for each of the fields on the registration form.

Thanks!
"Programming today is a race between software engineers striving to build bigger and better idiot-proof programs, and the Universe trying to produce bigger and better idiots. So far, the Universe is winning."

The Sporting Edge - Free NFL Football Pool


#10 underparnv

underparnv
  • Members
  • PipPipPip
  • Advanced Member
  • 30 posts
  • LocationReno, Nevada

Posted 10 August 2006 - 06:18 PM

So I'm trying to put in my football image for the rollover on the navigation, and I've used this code in the CSS:

#nav ul li a:hover {
    background: #798550 url(nav.jpg) no-repeat top;
}

Now...when viewing the site, the rollover is only the color, not the image.

What gives?!  Is it because it's the a tag and not the li???
"Programming today is a race between software engineers striving to build bigger and better idiot-proof programs, and the Universe trying to produce bigger and better idiots. So far, the Universe is winning."

The Sporting Edge - Free NFL Football Pool


#11 pixy

pixy
  • Members
  • PipPipPip
  • Advanced Member
  • 295 posts

Posted 11 August 2006 - 01:02 AM

You NEED more cellpadding on the sides. It's to cramped. I personally think you should widen the layout.

This is a .44 Caliber Loveletter straight through my heart.

Tabulas + Threadless + Hire Me!


#12 underparnv

underparnv
  • Members
  • PipPipPip
  • Advanced Member
  • 30 posts
  • LocationReno, Nevada

Posted 15 September 2006 - 04:11 PM

Alright everyone.  Instead of trying to add in padding to the container, and re-doing the nav, I decided to start from scratch.  This is how it turned out:

http://thesportinged...se20_screen.gif (98.48 KB)

This is another XHTML 1.0 Strict compliant design...and I am quite proud of it.

I would love to hear your comments!

Thanks.

underparnv
"Programming today is a race between software engineers striving to build bigger and better idiot-proof programs, and the Universe trying to produce bigger and better idiots. So far, the Universe is winning."

The Sporting Edge - Free NFL Football Pool


#13 Daniel0

Daniel0
  • Staff Alumni
  • Advanced Member
  • 11,956 posts

Posted 18 September 2006 - 06:55 AM

1) Make the username field as wide a the password field.
2) I don't like the button saying that it's valid XHTML Strict when it's not (it found 1 error when I validated it).
3) The same goes for the CSS (6 errors).
4) Use percentages instead of pixels. Users with low screen resolutions can't view it.
Edit: That was for the first layout.

Layout two (the posted screenshot):
It feels like there is something missing where it says "The Sporting Edge".

#14 steviewdr

steviewdr
  • Moderators
  • Advanced Member
  • 1,364 posts
  • LocationIreland

Posted 18 September 2006 - 10:24 AM

I see you are using opacity in the footer!!  ;) Aint work in Opera.

Seems ok apart from that.
-steve

#15 underparnv

underparnv
  • Members
  • PipPipPip
  • Advanced Member
  • 30 posts
  • LocationReno, Nevada

Posted 18 September 2006 - 03:19 PM

1) Make the username field as wide a the password field.
2) I don't like the button saying that it's valid XHTML Strict when it's not (it found 1 error when I validated it).
3) The same goes for the CSS (6 errors).
4) Use percentages instead of pixels. Users with low screen resolutions can't view it.
Edit: That was for the first layout.

Layout two (the posted screenshot):
It feels like there is something missing where it says "The Sporting Edge".


Hey Daniel0,

On the first layout, the only reason the site doesn't validate (the one error) is because when PHP inserts the hidden session id field, it puts it in right after the opening form tag.  I am looking for a fix for this now...however, I wasn't aware of the CSS errors.  I will get those fixed.

Also, how do you figure that users with lower screen resolutions can't see my site?  It looks perfect in 800x600...do I need to go any lower?

What do you feel is missing on the "The Sporting Edge" portion of the second design?

I see you are using opacity in the footer!!  ;) Aint work in Opera.

Seems ok apart from that.
-steve


The opacity thing isn't all that exciting.  I just put it in there because it looks cool.  It is gone in the second design, which is the reason for the bump on this thread.

Thanks!!

underparnv
"Programming today is a race between software engineers striving to build bigger and better idiot-proof programs, and the Universe trying to produce bigger and better idiots. So far, the Universe is winning."

The Sporting Edge - Free NFL Football Pool


#16 underparnv

underparnv
  • Members
  • PipPipPip
  • Advanced Member
  • 30 posts
  • LocationReno, Nevada

Posted 18 September 2006 - 07:05 PM

Just for the record, this is the design I would currently like reviewed (should I start a new thread?)

http://thesportinged...s.biz/version2/

Thanks!!
"Programming today is a race between software engineers striving to build bigger and better idiot-proof programs, and the Universe trying to produce bigger and better idiots. So far, the Universe is winning."

The Sporting Edge - Free NFL Football Pool


#17 steviewdr

steviewdr
  • Moderators
  • Advanced Member
  • 1,364 posts
  • LocationIreland

Posted 19 September 2006 - 01:22 PM

It looks ok. There is a lot of stuff there.
Your images seem to be broken in IE. You have images/objects like :D and :_PIT: which IE doesnt like.

Your contact form is obviously in development.

-steve

#18 underparnv

underparnv
  • Members
  • PipPipPip
  • Advanced Member
  • 30 posts
  • LocationReno, Nevada

Posted 19 September 2006 - 05:39 PM

Hey steviewdr,

Thanks for the feedback.  Yes, the contact form is still in development (as is a lot of things on the version2 site...).  The reason for the images being broken is that they are currently pointing to localhost...which I will get fixed  :D

When you say there is a lot of stuff there, what should be taken away?

Thanks!
"Programming today is a race between software engineers striving to build bigger and better idiot-proof programs, and the Universe trying to produce bigger and better idiots. So far, the Universe is winning."

The Sporting Edge - Free NFL Football Pool


#19 Daniel0

Daniel0
  • Staff Alumni
  • Advanced Member
  • 11,956 posts

Posted 19 September 2006 - 05:48 PM

Also, how do you figure that users with lower screen resolutions can't see my site?  It looks perfect in 800x600...do I need to go any lower?


Well, on very low screen resolutions you can view it, but it is kind of annoying having to scroll to the side, so if you use percentages instead of pixels the layout will fit according to the user's screen resolution.

What do you feel is missing on the "The Sporting Edge" portion of the second design?


I am not sure. A logo perhaps?

#20 underparnv

underparnv
  • Members
  • PipPipPip
  • Advanced Member
  • 30 posts
  • LocationReno, Nevada

Posted 19 September 2006 - 05:53 PM

Also, how do you figure that users with lower screen resolutions can't see my site?  It looks perfect in 800x600...do I need to go any lower?


Well, on very low screen resolutions you can view it, but it is kind of annoying having to scroll to the side, so if you use percentages instead of pixels the layout will fit according to the user's screen resolution.

What do you feel is missing on the "The Sporting Edge" portion of the second design?


I am not sure. A logo perhaps?


On the first design, I don't get a scroll on 800x600...do you?

I know that on the second one there is a scroll, but according to my stats, only 1% of my visitors use 800x600 anyway...and you can still see all the content without scrolling left-right, so I'm not too worried about it...

Logo?!?!  I am a DEVELOPER...not a designer!!  :P
"Programming today is a race between software engineers striving to build bigger and better idiot-proof programs, and the Universe trying to produce bigger and better idiots. So far, the Universe is winning."

The Sporting Edge - Free NFL Football Pool





0 user(s) are reading this topic

0 members, 0 guests, 0 anonymous users