SITE REVIEW: Small Sports

Hello, Can you guys and girls please review http://www.smallsports.com

I know its php-nuke but i have made most of my blocks and modules and graphics to. Also, check out the forum. I have put a lot of work into it with tons of hacks i have added to it. so check that at to.

http://www.smallsports.com

[Using Mozilla Firebird]

Hmm, let's start with the positioning since it's the biggest thing that stands out right now. It might help if the title had repeating images to the left and right of the title and it was centered, it just looks odd off to the left (I'm in 1280x1024 resolution btw). Also, in your news posts, there's a BIG gap with nothing to the right of the images in every post. Aligning the text to go in that gap rather than the bottom of the image would help. Also add some margins to the images and text.

The forums preview script up top in my opinion is pretty pointless (even though it is really cool). I'd leave that to the users to look through the forums if they want to see what topics are being talked about. Sometimes you get the odd post like "Hey, I'm new here" in the script which I doubt is something someone new to the site wants to see when they first see the site.

Color Scheme (I comment on color sheme on everyone's site just because I've done a lot of work in that area and know my color wheel and find it to be really important): Awesome work. I'm not personally a fan of that color yellow for the headers, but it does match with the dark red and gray color scheme.

Overall: I really just think it needs some work with space management, the rest looks great. And nice work on the modules btw even though I'm also not a fan of php-nuke.

dude, it looks weird because it was designed in 800x600 and its best viewed in that so i dont know what it looks in a smaller res

I'm just saying you can add dynamic sizing tables on both sides of the title graphic and it'll look the same in 800x600 while resizing itself to look good in 1024x768, 1280x1024, and even 1600x1200 if it came down to it. Or just have one on either side and make it align to the left or right.



That would center it. It could also be done using CSS/DIV, but I see your using tables anyway. I also just noticed that you do have the HTML exactly like that, but your missing the background image (I get a 404 File Not Found on http://www.smallsports.com/themes/subSy ... ges/bg.gif , supposedly where the HTML says it is, probably why it isn't doing what I'm suggesting)

see i added that image and it looks really bad now. i added the bg.gif and it looks really really bad

no, that's perfect, now try this to fix the mis-alignment:

instead of this:



try this:



now, two things to note in the changes.

1. valign="top" : this will align the image to the top of the table (as it looks like it's not aligning cuz the image isn't the same height as the background)
2. style="margin-top: 0px;" : this currently does nothing right now! but, if for some reason the image needs to be moved down to align with the background, keep adding 1px to the value.

BTW, I like the new right sidebar addition.

ok, Thanks for that. That is fixed and actually looks pretty nice. Now what

doh, that's funny that it worked cuz I made a typo.

change style:"margin-top: 0px;" to style="margin-top: 0px;"

other than that, add "align="right"" to all you image tags for all the 200x300 news images.

that's all I can advise on at that point, the rest is up to your preference.

u want me to change the typo?

Yes change it Minne. I think that typo actually causes problems on some browsers. Some browsers are smart enough to understand what you really meant. I recall doing something like that in the past and I think it worked fine in IE, but not Netscape or Opera. Not sure about this case though, but you should change it.

ok, i did that. i just added a new banner, and the bg.gif is being weird again. help

remember the style="margin-top: 0px;" that I had you put in? it looks like you cut more off the top of the banner, so all you need to do is change it to 4px, 5px, 6px, 7px, or more to fix it. You'll get the idea how it works when you play with it.

ok,I am having a little trouble. i went all the way to 13 and it wont turn out right. its at 8 right now

also, when i align pic right. the pic goes over the text and looks bad.

if u could, give me the code with the size 200x300 and the align to see if it works with your code.

also need to get top pic fixed

or try style="positioning: absolute; top: 0px;"

where he told you to switch before.