My new CSS only site

[Goodings Media]

Hiya guys,

I remade my www.GoodingsMedia.com site. The old design looked shit...

Im not actually a web designer/coder, I just like the 'business side of things' in this game, but this time I thought I'd give it a go.

So through a lot of trial and error and photoshopping and google searches, heres the result

http://www.goodingsmedia.com

I havn't finished all the pages yet. Need to type up the terms, make the login form, sort out a knowledgebase and a contact form. But its 5.47am and I'm tired....

Let me know what you think

The main purpose is to sell webspace (for blogs, small tgps, link dumps etc) and to promote my own sites.

Thanks

[donborno]

Some "technical" comments:

• In link tags, there is no target attribute
• In your portfolio: Trade traffic/Trade links etc. - those are no paragraphs, it's rather a list.
• On your hosting page: those are no paragraphs either, it's clearly a table (role of thumb: use a table as soon as you find a proper header for each column)
• Don't use a border attribute for images. Border effects the design, so it's CSSs job
• In your footer, you don't end the p-tag, but use <br /> - I'd recommend adding #foot p {padding-bottom: 1em} instead
• You should wrap your content in a div box - a div box is designed to combine those elements which belong together, and your content definitely does
• IE will show an image's alt attribute as its title, as long as no title is defined. So add title="" to your images, it links stupid if you move your mouse on a logo and a caption "logo" pops up

In terms of design I like it, unless I think you waste quite a lot of space above the header, which makes it difficult to focus on your content.

[PinPine]

Well nice job! I'm the same as you, I also know the basics when it comes to photoshop etc. But I think your site has a nice and easy on the eye design.

[Goodings Media]

Some "technical" comments:

• In link tags, there is no target attribute
• In your portfolio: Trade traffic/Trade links etc. - those are no paragraphs, it's rather a list.
• On your hosting page: those are no paragraphs either, it's clearly a table (role of thumb: use a table as soon as you find a proper header for each column)
• Don't use a border attribute for images. Border effects the design, so it's CSSs job
• In your footer, you don't end the p-tag, but use <br /> - I'd recommend adding #foot p {padding-bottom: 1em} instead
• You should wrap your content in a div box - a div box is designed to combine those elements which belong together, and your content definitely does
• IE will show an image's alt attribute as its title, as long as no title is defined. So add title="" to your images, it links stupid if you move your mouse on a logo and a caption "logo" pops up

In terms of design I like it, unless I think you waste quite a lot of space above the header, which makes it difficult to focus on your content.

mate thats awesome! exactly the type of critisms I need

Thanks, will be off to fix things.

Pinpine - thanks for the kind words