SITE REVIEW: Small Sports

  • Minne
  • Student
  • Student
  • No Avatar
  • Joined: Jun 21, 2003
  • Posts: 94
  • Loc: Small Sports
  • Status: Offline

Post August 27th, 2003, 7:28 pm

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
  • Anonymous
  • Bot
  • No Avatar
  • Joined: 25 Feb 2008
  • Posts: ?
  • Loc: Ozzuland
  • Status: Online

Post August 27th, 2003, 7:28 pm

  • tierra
  • Student
  • Student
  • User avatar
  • Joined: Aug 29, 2003
  • Posts: 91
  • Status: Offline

Post August 30th, 2003, 7:33 am

[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.
  • Minne
  • Student
  • Student
  • No Avatar
  • Joined: Jun 21, 2003
  • Posts: 94
  • Loc: Small Sports
  • Status: Offline

Post August 30th, 2003, 9:46 am

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
  • tierra
  • Student
  • Student
  • User avatar
  • Joined: Aug 29, 2003
  • Posts: 91
  • Status: Offline

Post August 30th, 2003, 12:44 pm

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.

Code: [ Select ]
<table width="100%">
<tr>
<td background="myrepeatingimage.jpg">
<td><img src="mytitle.jpg">
<td background="myrepeatingimage.jpg">
</tr>
</table>
  1. <table width="100%">
  2. <tr>
  3. <td background="myrepeatingimage.jpg">
  4. <td><img src="mytitle.jpg">
  5. <td background="myrepeatingimage.jpg">
  6. </tr>
  7. </table>


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)
  • Minne
  • Student
  • Student
  • No Avatar
  • Joined: Jun 21, 2003
  • Posts: 94
  • Loc: Small Sports
  • Status: Offline

Post August 30th, 2003, 2:35 pm

see i added that image and it looks really bad now. i added the bg.gif and it looks really really bad
  • tierra
  • Student
  • Student
  • User avatar
  • Joined: Aug 29, 2003
  • Posts: 91
  • Status: Offline

Post August 30th, 2003, 3:04 pm

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

instead of this:

Code: [ Select ]
    <td width="60%" height="100" bgcolor="#ADADAD" background="themes/subSystem/images/bg.gif"><a href="index.php"><img src="themes/subSystem/images/logo.gif" border="0" alt="Welcome to Small Sports"></a></td>
    <td width="40%" height="100" bgcolor="#ADADAD" background="themes/subSystem/images/bg.gif"></td>
  1.     <td width="60%" height="100" bgcolor="#ADADAD" background="themes/subSystem/images/bg.gif"><a href="index.php"><img src="themes/subSystem/images/logo.gif" border="0" alt="Welcome to Small Sports"></a></td>
  2.     <td width="40%" height="100" bgcolor="#ADADAD" background="themes/subSystem/images/bg.gif"></td>


try this:

Code: [ Select ]
    <td width="60%" height="100" bgcolor="#ADADAD" background="themes/subSystem/images/bg.gif" valign="top"><a href="index.php"><img src="themes/subSystem/images/logo.gif" border="0" alt="Welcome to Small Sports" style:"margin-top: 0px;"></a></td>
    <td width="40%" height="100" bgcolor="#ADADAD" background="themes/subSystem/images/bg.gif"></td>
  1.     <td width="60%" height="100" bgcolor="#ADADAD" background="themes/subSystem/images/bg.gif" valign="top"><a href="index.php"><img src="themes/subSystem/images/logo.gif" border="0" alt="Welcome to Small Sports" style:"margin-top: 0px;"></a></td>
  2.     <td width="40%" height="100" bgcolor="#ADADAD" background="themes/subSystem/images/bg.gif"></td>


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.
  • Minne
  • Student
  • Student
  • No Avatar
  • Joined: Jun 21, 2003
  • Posts: 94
  • Loc: Small Sports
  • Status: Offline

Post August 30th, 2003, 3:08 pm

ok, Thanks for that. That is fixed and actually looks pretty nice. Now what
  • tierra
  • Student
  • Student
  • User avatar
  • Joined: Aug 29, 2003
  • Posts: 91
  • Status: Offline

Post August 30th, 2003, 4:09 pm

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.
  • Minne
  • Student
  • Student
  • No Avatar
  • Joined: Jun 21, 2003
  • Posts: 94
  • Loc: Small Sports
  • Status: Offline

Post August 30th, 2003, 7:46 pm

u want me to change the typo?
  • Bigwebmaster
  • Site Admin
  • Site Admin
  • User avatar
  • Joined: Dec 20, 2002
  • Posts: 8925
  • Loc: Seattle, WA & Phoenix, AZ
  • Status: Offline

Post August 30th, 2003, 8:17 pm

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.
Ozzu Hosting - Want your website on a fast server like Ozzu?
  • Minne
  • Student
  • Student
  • No Avatar
  • Joined: Jun 21, 2003
  • Posts: 94
  • Loc: Small Sports
  • Status: Offline

Post August 31st, 2003, 7:51 pm

ok, i did that. i just added a new banner, and the bg.gif is being weird again. help
  • tierra
  • Student
  • Student
  • User avatar
  • Joined: Aug 29, 2003
  • Posts: 91
  • Status: Offline

Post August 31st, 2003, 8:02 pm

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.
  • Minne
  • Student
  • Student
  • No Avatar
  • Joined: Jun 21, 2003
  • Posts: 94
  • Loc: Small Sports
  • Status: Offline

Post August 31st, 2003, 8:12 pm

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
  • Minne
  • Student
  • Student
  • No Avatar
  • Joined: Jun 21, 2003
  • Posts: 94
  • Loc: Small Sports
  • Status: Offline

Post August 31st, 2003, 8:34 pm

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
  • b_heyer
  • Web Master
  • Web Master
  • User avatar
  • Joined: Jun 15, 2003
  • Posts: 4583
  • Loc: Maryland
  • Status: Offline

Post August 31st, 2003, 8:36 pm

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

where he told you to switch before.
Pixel Acres V2
  • Anonymous
  • Bot
  • No Avatar
  • Joined: 25 Feb 2008
  • Posts: ?
  • Loc: Ozzuland
  • Status: Online

Post August 31st, 2003, 8:36 pm

Post Information

  • Total Posts in this topic: 33 posts
  • Moderator: Website Reviewers
  • Users browsing this forum: No registered users and 39 guests
  • You cannot post new topics in this forum
  • You cannot reply to topics in this forum
  • You cannot edit your posts in this forum
  • You cannot delete your posts in this forum
  • You cannot post attachments in this forum
 
 

© 2011 Unmelted, LLC. Ozzu® is a registered trademark of Unmelted, LLC.