SITE REVIEW: Small Sports

  • Minne
  • Student
  • Student
  • Minne
  • Posts: 94
  • Loc: Small Sports

Post 3+ Months Ago

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
  • Posts: ?
  • Loc: Ozzuland
  • Status: Online

Post 3+ Months Ago

  • tierra
  • Student
  • Student
  • User avatar
  • Posts: 91

Post 3+ Months Ago

[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
  • Minne
  • Posts: 94
  • Loc: Small Sports

Post 3+ Months Ago

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
  • Posts: 91

Post 3+ Months Ago

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
  • Minne
  • Posts: 94
  • Loc: Small Sports

Post 3+ Months Ago

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
  • Posts: 91

Post 3+ Months Ago

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
  • Minne
  • Posts: 94
  • Loc: Small Sports

Post 3+ Months Ago

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

Post 3+ Months Ago

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
  • Minne
  • Posts: 94
  • Loc: Small Sports

Post 3+ Months Ago

u want me to change the typo?
  • Bigwebmaster
  • Site Admin
  • Site Admin
  • User avatar
  • Posts: 9086
  • Loc: Seattle, WA & Phoenix, AZ

Post 3+ Months Ago

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.
  • Minne
  • Student
  • Student
  • Minne
  • Posts: 94
  • Loc: Small Sports

Post 3+ Months Ago

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

Post 3+ Months Ago

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
  • Minne
  • Posts: 94
  • Loc: Small Sports

Post 3+ Months Ago

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
  • Minne
  • Posts: 94
  • Loc: Small Sports

Post 3+ Months Ago

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
  • Posts: 4581
  • Loc: Maryland

Post 3+ Months Ago

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

where he told you to switch before.
  • She God
  • Student
  • Student
  • User avatar
  • Posts: 72
  • Loc: California (Land Of The Cackling Sun)

Post 3+ Months Ago

Your overall layout is actually rather nice, but the top title image, and the background do not blend together at all (But then again you might just be working on it ) There’s only one thing I could complain about...and that’s mostly due to personal taste. Gray? I’ve always disliked gray. When I think of sports I think something that’s fast and fun. A swirl of colors rather then just the monotone of using gray. Red and yellow look good, but then you throw in that blue and it throws me off.

Like I said it’s more personal taste, so good job on the whole.
  • Minne
  • Student
  • Student
  • Minne
  • Posts: 94
  • Loc: Small Sports

Post 3+ Months Ago

nope, look still like the same.

also if u could, give me the code with the size 200x300 and the align to see if it works with your code.
  • tierra
  • Student
  • Student
  • User avatar
  • Posts: 91

Post 3+ Months Ago

well, your news is obviously generated by either php-nuke or whatever else, so you'll have to put in the code into your settings, but for example:

This is what you currently have:

Code: [ Select ]
<table width="100%" border="0" cellspacing="0" cellpadding="4">
 <tr>
  <td><a href="modules.php?name=News&new_topic=3">
  <img src="images/topics/Minne.png" border="0" alt="Minne" title="Minne" align="right" hspace="10" vspace="10">
  </a><font class="storycontent"><img src="http://www.smallsports.com/koren.jpg" length=200 width=300">
  <br>The Seattle Seahawks are playoff bound. Tkae my word. I am saying it from the start. Its not
  because I am from Seattle but I have this little gut feeling in my tummy.
  </font>
  </td>
 </tr>
</table>
  1. <table width="100%" border="0" cellspacing="0" cellpadding="4">
  2.  <tr>
  3.   <td><a href="modules.php?name=News&new_topic=3">
  4.   <img src="images/topics/Minne.png" border="0" alt="Minne" title="Minne" align="right" hspace="10" vspace="10">
  5.   </a><font class="storycontent"><img src="http://www.smallsports.com/koren.jpg" length=200 width=300">
  6.   <br>The Seattle Seahawks are playoff bound. Tkae my word. I am saying it from the start. Its not
  7.   because I am from Seattle but I have this little gut feeling in my tummy.
  8.   </font>
  9.   </td>
  10.  </tr>
  11. </table>


Try putting this as the 200x300 img tag instead:

Code: [ Select ]
<img src="http://www.smallsports.com/koren.jpg" length=200 width=300" align="left">


Ok, as for the banner, I fixed it all up for you, download this and put it up: http://tierra.gnuk.net/minnelogo.gif

I had to crop off the top of his head because it was already cropped a little on the top from the original, so I just made it look like he's only in the back of the image.
  • Minne
  • Student
  • Student
  • Minne
  • Posts: 94
  • Loc: Small Sports

Post 3+ Months Ago

ok top problem fixed. You think the seahawk article looks good like that? Because then the topic picture is in the way.
  • tierra
  • Student
  • Student
  • User avatar
  • Posts: 91

Post 3+ Months Ago

Well, I really think you should do that AND make all news pictures smaller than the 200x300... maybe more like 100x150 and if the user wants to see a bigger picture, have the thumbnail link to the bigger picture.
  • Minne
  • Student
  • Student
  • Minne
  • Posts: 94
  • Loc: Small Sports

Post 3+ Months Ago

jusr added it like that. You think it looks better? if it makes the main page better then i should do it
  • tierra
  • Student
  • Student
  • User avatar
  • Posts: 91

Post 3+ Months Ago

yeah, I like that better... again, this is all up to personal preference, so it's completely up to you whether you want to do it or not, but I personally think it looks better that way (and it'll stand out a lot more if done to all the posts).
  • Minne
  • Student
  • Student
  • Minne
  • Posts: 94
  • Loc: Small Sports

Post 3+ Months Ago

ok, can u add the code to it real quick so when they click it, it goes bigger
  • tierra
  • Student
  • Student
  • User avatar
  • Posts: 91

Post 3+ Months Ago

well, I meant just link the 100x150 image to the 200x300 image, but if you want to do that:

Code: [ Select ]
<img src="mysmall.jpg" onMouseClick="this.src=='mybig.jpg'?this.src='mysmall.jpg':this.src='mybig.jpg';" align="left">


I think that'll work... that's off the top of my head...
  • Minne
  • Student
  • Student
  • Minne
  • Posts: 94
  • Loc: Small Sports

Post 3+ Months Ago

Ok, I am starting to add the lil pics. You think i should add more articles to front page. i can configure how much articles are on there at a time
  • tierra
  • Student
  • Student
  • User avatar
  • Posts: 91

Post 3+ Months Ago

well, if you were to change it, maybe go down to 10, but it looks like news gets posted often enough that maybe 15 is fine.
  • Minne
  • Student
  • Student
  • Minne
  • Posts: 94
  • Loc: Small Sports

Post 3+ Months Ago

Its Already At 15...... :shock:

You think I should change it down, leave it?

Also, whats next on the agenda to fix?
  • tierra
  • Student
  • Student
  • User avatar
  • Posts: 91

Post 3+ Months Ago

yeah, 15 is fine, leave it.

well, that's all I thought really needed changing, looks great now.
  • compuboard
  • Born
  • Born
  • compuboard
  • Posts: 3

Post 3+ Months Ago

i like the graphics youve done and youve done a good job with the hacks.

8/10 i would of given 10/10 but u cheated a bit using php nuke lol :D
  • Minne
  • Student
  • Student
  • Minne
  • Posts: 94
  • Loc: Small Sports

Post 3+ Months Ago

Ya, i did use nuke. But i have learned to customize it. make my own blocks. make my own modules to fit my needs for my site
  • Anonymous
  • Bot
  • No Avatar
  • Posts: ?
  • Loc: Ozzuland
  • Status: Online

Post 3+ Months Ago

Post Information

  • Total Posts in this topic: 33 posts
  • Moderator: Website Reviewers
  • Users browsing this forum: No registered users and 9 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
 
 

© 1998-2014. Ozzu® is a registered trademark of Unmelted, LLC.