Working ok for me. Congrats on getting your first affiliate site beyond the drawing board stage. First impressions are along the lines of the others - no need to have the frames - you can easily centre the page without frames by using the following css:
---------------------------------------------
body {
text-align:center;
}
#wrapper {
width:970px; (or whatever your overall page width is)
margin:0px auto;
text-align:left;
}
in your html add a wrapper div around all your content:
HTML Code:
<div id='wrapper'>
all your page content goes within here
</div>
----------------------------------------------
You could slim your pages down a lot if you put all the css in a seperate stylesheet as you've had to reapeat lots of the same styling to each div.
Think the 'Beauty Solutions Review' text on the photo in the top right only obscures what is a good image and the text would be better placed under the main title as a strapline.
I also feel that the dark grey lines dividing the page are a bit heavy handed and could either be a much lighter grey or removed altogether. I don't think you'd need them if you added a bigger margin/more paddng to the content area.
On the make up ranges it took me a while to realise I had to click on the image for more info - a text link or button leading to the range would be clearer.
Hope some of this helps, best of luck with it.
LinkBack URL
About LinkBacks
Reply With Quote
)
Originally Posted by warren123
)
Bookmarks