REVIEW REQUEST: The Nukesilo

Oh, and btw, dont take any notice of my source code

hmmm.... What game is this for? I'm guessing command & conquer... You don't say anywhere... I know people how go looking for your site will know what its for but what about search engine? if I searched for command & conquer would your site come up?
I think the black & orange work but it is very dark... One thing I have noticed is that your top logo/banner keeps moving and changing the order in which its displayed... Is it meant to do that? Also when it does do that it breaks the grey borders you have and they just look jumbled...
Lack of a distinct homepage... At the moment when you first get into the site you're at the news page... Maybe you should have an introduction page with more about the game and what your site offers... (Just my opinion)
When you said don't look at the source code thats the first thing I did...
And omg... I'm guessing you're using some kind of ssi but wtf is with every bit of text having the font tag... I saw you've already got a style sheet so just move all that info into the style sheet... It'll still be called when you ssi so it shouldn't be a problem... If you want to see some sites that explain how to move alot of the tags into CSS I can find some of the sites that I used...
Overall I like the layout but you've gotta get your tags fixed...

Hope this was some help

DavidJaymz

The first thing I notice is the two graphics beside the logo graphic. Those are very eye-catching images but I feel that they don't coordinate very well with each other at all. The right one in particular, with the green and white, does not work well with the rest of the page design. I would do just a simple hue/saturation fix to get the colours to match with everything else.

Now I'm noticing that there are three little pictures underneath the "Nukesilo" text but I can't quite make out what thye are. The middle one looks like a dog, the right one looks like a man playing a musical isntrument, but I can't see anything in the left one. Something needs to be done about those.

The third thing I noticed is that there is a heck of a lot of scrolling on this page. Some of those news items in the body area should be moved to an archives page at some point. Also try to find a way to shorten up that menu a little bit.

The fourth thing I'm noticing is that the font size here is way too small for me. Trying to read it is giving me a headache. Size="1" is way too small for text of any importance.

I do find that you have done a good job with the colours and visual look and feel, although I do agree that it is a little on the dark side.

Quote: Originally posted by davidjaymz
When you said don't look at the source code thats the first thing I did...

*raises hand*

Me too! Then I decided that it was way too early to try and make sense of that. It reminds me of an early site I did, one of my first times using css. It wasn't until I was done that I discovered that my client was using Netscape3 and wasn't gonna upgrade. It was quite a large site and took some inventive global search & replacing to get it working in his browser. The resulting code was a nightmare!

On to this site...

I agree with all that's been said. In addition:

My biggest problem is with your in-page links. The link colour is almost indestinguishable from your text colour and have no text decoration to identify them. Compounded with this is the fact that you have all kinds of underlined (and bold) segments that DO look like links.

Hi Nuke,

Just a few comments....
I think a solid background for the main body would be better than the grid background. Text would be easier to read.

The links on the left and the affiliates and counter on the right are jammed up against my screen. (IE 6.0) Looks fine in NS 4.79 and Opera 6....just need a wee bit of padding in those table cells for IE. Pad that middle table too.

Noticed that your alt text for the logo at the top was alt="Just the toplogo". You can utilize your alt text by using keywords/phrases there. Change your alt text to something that describes your site. Maybe "Nukesilo: Games and cheats for serious gamers" (or something to that effect) I am not a gamer so you could probably come up with something better.

On the other pages, the images at the very top are not in the same order as the home page.

You need a "Home" page link. You can also make your top logo clickable and point it back to the home page.

I wasn't expecting such a great reply, I didn't know so many things were wrong, I'll try and fix them all up.

I've tried using css for the fonts/colours/etc, but got nowhere, maybe if anyone knows of a few tutorials that could help me?

Thanks again for the comments

http://www.webmonkey.com and http://www.w3schools.com both have good CSS tutorials (but don't try w3schools with Opera - their font size is miniscule in that browser for some reason - unless you want to zoom to 150% to be able to read it!)

Thanks for that, I'll check that site out..

add marginheight="0" marginwidth="0" to your body tag to make the body margind zero in NS and Opera as well

you do need to fix your code, you have 2 and tags.

as for the layout colors, I think its way to orange in IE, NS4.7 is fine as the table borders are grey (default) but links are orange. you have a lot of content on there, and a lot of links, so i guess saying break it a bit would be pointless lol. but try use more of the center section, making that wider will make your page shorter.

Quote:
but try use more of the center section, making that wider will make your page shorter.

What do you mean by that?

I think what Busy means is if you would make the center part (your main body) wider then you wouldn't have the long, long, long center part that cause you to have to scroll and scroll and scroll. Making it wider will shorten the page some.

Ouch! You have tons of coding errors.
Get rid of one of your doctypes at the top.
Your style sheet should be

Just below your there is some extra stuff (in red) you can get rid of.

Menu

The first table tag just below needs to be closed.

Just above where you have "Vote For My Site:" there is an extra and . You can get rid or these.

Really, there are lots more errors, especially with yout and and some font tags too. have you looked at your site using a code validator? Would be a good idea if you did. This would help to point out any mistakes that you have. You can correct and then recheck your work until you get things squared away. I recommend netmechanic.com .
Go there and look at box #1 where it says, " Fix HTML Code Errors." and click on "TRY IT".

I noticed that you still have loads of font tags too. Are you going to get rid of those and put your nifty style sheet to use? Did the tutorials that Megan gave you help any?

Yell if you need a kick start.

You know nuke, actually I think you need to start from scratch with your tables. They are absolutely a mess.....like sorta beyond repair. LOL

I copied your code and took it to my Dreamweaver and nothing showed up except for the left hand side menu. It would probably be easier to fix this if you just started over rather than trying to go in there and find what 's and 's you are missing or which ones you do not need. Your page is showing up fine in Internet explorer but if you don't fix these mistakes it will only cause you heartache in the future. Believe someone who has been there done that.

I dont know where to start, so much to do, so little time....

I had a look at Netmechanic and the amount of errors it came up with was unbelieveable. I think I better start again from scratch.
And your right about the missing tags disater-master.
My source code is a horrible mess!! :eek:

I'll redo my site using the aid of everyones' comments and suggestions,
and when I return, there wont be any bad things you'll be saying about my site

Your help is much appreciated,
Nuke

You are welcome. And don't forget to post here when you get the other one going. There is always plenty of help if you need it.

Good luck!

I fixed the toplogo, it doesn't move around and there are no grey borders for it to break.

I neatened and fixed up my source code, and I hardly have any font tags. The source is has more comments.

I fixed up the graphic on the right of the toplogo, it now suits the page better.

Those pictures underneath the text "The Nukesilo" on the toplogo have been left the way they are. People who play the game know what they are, and can recognise them..

I've reduced the amount of news on the page by half, and I've shortened that menu.

I've kept the size 1 text. If you cant read it, theres an easier solution, change the text size on your broswer.
(view - text size on menu) The font 2 looks way too big on a computer with an 800X600 resolution set.

Hopefully the links are now distinguishable from the rest of the text, they are now a whole lot lighter and hopefully stand out.

The toplogo is now clickable which takes you to the home page, and there is also a text link on the menu that takes you home.

I have added some descriptive text to my toplogo with tells visitors a little about my site.

I have added some cell padding everywhere so hopefully everything looks ok in that department.

I have fixed up the css/style probs and have fixed all missing/shouldn't be there "td's" and "tr's"

-----------------------------
All this was done to my main/index page, I'd just like a few people to have another look over to see if anything is stuffed, out of place, doesn't look right, etc..

Thanks allot,
Nuke

Hmm, a note about the links: the links in the body section have an underline which disappear on hover. But visited links no longer show an underline, nor do they underline on hover. Consider making all links, visited or not show an underline. Or else they get lost within the text (colors are similar).

Change your heading font color, having them white or something will make them stand out more
The font size is good on my pc at 800x600
when you use >> in your pages, use >> using things like > can confuse older browsers if your code isnt up to stratch.
you could use more CSS if you want, to get rid of all those font tags, if you add class="??" into your td cell, that whole cell will be affected, if you have things in there you want different, use the span tag

I agree with Busy that using white as a heading colour will make things stand out more as right now I think the orange is a bit too much.

I was trying to work out why some links were underlined and other weren't. I just couldn't work it out
Thanks for that Abhishek, I'll fix it right away.

Do you mean all the headings, the headings on the news?? Which ones?? Also, could you give me the code for using the class tag with table cells, that would be very handy! Then all I would have to do is span around headings and not the text.. :coolblock

Thanks

All your headings, content headings.
In your CSS file add td {your font size, color ...};
and you dont need to add anything in the td cell, unless you only want to do one or two cells, then make a class, such as this in your CSS file:
.tdcell {your font size, color ...};
and in the td cells you want to be effected, add class="tdcell" to it.

Thanks for the info, do you think white headings would look better or white-grey headings?

I still need to class the headings. If I apply the css to my td cell, will the headings take on that, or will they do what there told, because I dont want really small headings!

I have recently switched over to a new layout, I was wondering what everyone thinks of it on the outside and is my html alright on the inside?

Go HERE to see my site.

Thanks,
Nuke

I'm getting a very small horizontal scrollbar at 800*600 in IE5.5 & NS6.2 and in NS4.7 only the top graphic is showing. The top part of the navigation graphic looks a bit odd with the little green bit on the top. Not overly keen on dark sites but I quite like these colours!

I do want to make my site as broswer friendly as possible, I wonder what could be causing it not to show? Any Ideas?

I'll get rid of that scroll bar right away, and that top bit on the navigation graphic is as good as gone. I didn't like it anyway.

If I remember the old site correctly, this new one is a big, huge improvement.

My current layout is a big improvement from the last one!

try pad the interactive links off the right side a bit, if you cant pad the cell use a   at the end of the linked word.

on the left side, the redalert2 and site info, it seems your tags have changed, font is way bigger in Opera6

ok in Ns4.7 except iframe doesnt show

I'll do a bit of padding on the links and i'll have a look at the menu..