Originally posted by fred123go:
Ok finally, the width has now been narrowed so it fits every screen resolution perfectly, everything is HTML now as well.
I have done a few good reviews atm, an iMac one, a Altec Lansing one and a iPod Shuffle reviews -loads more coming. Also the family tree is basically complete - check it out!
Just gotta add more reviews, publish more Opinions and finish off the remaining Extras at Macinfo will be complete, well new stuff added weekly
Any more comments and pointers? Any would be really appreciated 
Oh sorry, was meant to post this in my original thread
I like what I see so far as regards content and the design is pretty cohesive so far. A little too much Apple-influenced, but I guess that's the point. So, I advise about general website-making goodness.
1 - The way you give the content boxes the rounded edges needs a bit of fixing. At the moment, you use such code as:
Code:
<td valign=middle><img src="Resources/item3b.gif" border=0 width=13 height=498 alt=""></td>
To define the left and right sides of the box. The image doesn't need to be 13x498 pixels in size. 13x1 will do fine. All you need to do is set the image as the background image for the table cell and tell it to repeat vertically. This has the added advantage of, when you increase font size in the browser, the white bars extend all the way up both sides of the box. At the moment, if you make the text too large, ugly gaps appear. So, I would use code like this:
Code:
<td style=''background-image: url(Resources/item3b.gif); background-repeat: repeat-y;" valign="middle">
<img src="Resources/item3b.gif" width="13" height="100%" border="0" alt=""><br>
</td>
A little bit of redundency, the image being used as both the table cell background image
and the contents of the cell as well, as you do now, just in case the background image doesn't work for some reason (surely you want to cater to Netscape 4.x users, yes?).
For good examples of the corner thing, see stuff I did a long long time ago;
megasad2,
megasad3 and
personal1.
2 - Use lower case for everything you can, especially file and directory names. It just makes life easier when writing out links / image references. Naming files so that you know what they are without having to open them is also a good thing.
3 - The black buttons at the bottom of the pages? The text inside of them would look better centred vertically. Also, use the same buttons on all pages, that the navigation is consistent. At the moment, you have smaller buttons at the bottom of the iPod shuffle review, larger at the bottom of the main page. Is for a reason?
4 - General obviousness; if you can be bothered, try and use CSS, once central stylesheet file, for all the font faces/sizes etc., rather than font tags. It makes life much easier when you want to change fonts across the whole site. For example, 949 pages of my
comic book library all use the
one CSS file. And XHTML is so damn similar to HTML there's no reason to not use that either.
Valid, mind.
That's all I can think of right now. I'm assuming you're hand coding this website...
Top
