Go Back   CodingForums.com > :: Client side development > General web building > Site reviews

Before you post, read our: Rules & Posting Guidelines

Reply
 
Thread Tools Rate Thread
Enjoy an ad free experience by logging in. Not a member yet? Register.
Old 02-06-2009, 09:33 PM   PM User | #1
x1ngxang
New to the CF scene

 
Join Date: Jan 2009
Posts: 2
Thanks: 1
Thanked 0 Times in 0 Posts
x1ngxang is an unknown quantity at this point
Macintosh Need honest opinions, brutal or otherwise ;)

Hey Dev Pros, I just launched this site and wanted to know your honest opinons about it. I'm a relative newbie developer, and set this site up for some friends of mine. I'm looking for guidance in what I could do to make it better. I'd like to make sure it looks good enough for visitors to buy from it! Easy to get around is good too. I'll welcome any suggestions as to what I could do to make it better too.

No comment is too brutal! LOL

~~x1ngxang~~
x1ngxang is offline   Reply With Quote
Old 02-06-2009, 10:00 PM   PM User | #2
Millenia
Open Source Zealot

 
Join Date: May 2008
Location: Lost in Localhost...
Posts: 702
Thanks: 3
Thanked 43 Times in 42 Posts
Millenia is on a distinguished road
Well, if you want true honesty:
It's coded wrong.

You have used tables for layout, and you should know that that is completely bad! Why Tables for Layout are Stupid!

You have 150 Errors, 232 warning(s) in your XHTML Markup. Including a missing Doctype!
You have 46 Errors in your CSS.

I find your whole design very amateurish looking. The images just look really nasty.

In my browser, the following is out of place on the right hand side table. The 'Links and Resources', the 'Ask a Golf Pro', the 'Quick Search' and finally the copyright notice.
I am using FF3 in Linux with a 1024x768 resolution.

Your page seems to load quite slow, and I predict it's your total overload with images. To speed it up a bit, you might want to use a CSS hover for the hover effect on your menu. Instead of Javascript (Excuse me if I'm wrong about that).

That's it for now.
Millenia is offline   Reply With Quote
Users who have thanked Millenia for this post:
x1ngxang (02-06-2009)
Old 02-06-2009, 10:23 PM   PM User | #3
neonflorida
New Coder

 
Join Date: Nov 2008
Posts: 21
Thanks: 3
Thanked 0 Times in 0 Posts
neonflorida is an unknown quantity at this point
Some of the right hand side links are off to the side. I am also using Linux and using a 1280 x 1024 resolution.

I am not a master programmer or anything close to it, but using tables is a big no no when designing the layout of a site.

Use the HTML validator that W3C offers. They also have one to validate CSS.
neonflorida is offline   Reply With Quote
Old 02-06-2009, 11:37 PM   PM User | #4
x1ngxang
New to the CF scene

 
Join Date: Jan 2009
Posts: 2
Thanks: 1
Thanked 0 Times in 0 Posts
x1ngxang is an unknown quantity at this point
@Millenia, thanks for the input. I'll look into those errors and redesign the layout to fix the table no-no.

@NeonFlorida - Thanks I'll look into that resource.

Keep it coming! Thanks!!
x1ngxang is offline   Reply With Quote
Old 02-07-2009, 01:28 AM   PM User | #5
demtron
Regular Coder

 
Join Date: Jun 2008
Location: SE Wisconsin, US
Posts: 222
Thanks: 1
Thanked 20 Times in 20 Posts
demtron is on a distinguished road
I am not seeing the same problems in FF 3 for IE. I think the hover-over images in the upper menu are blurry and unreadable as a higher resolution than 1024x768.
__________________
Milwaukee Web Designer and SEO Milwaukee Firm specializing in ASP.Net, C#, VB.Net, SQL Server and Access.
demtron is offline   Reply With Quote
Old 02-11-2009, 06:39 AM   PM User | #6
JulieV
Regular Coder

 
Join Date: Dec 2008
Location: Always Roaming Around Burbank Illnois or TO
Posts: 177
Thanks: 13
Thanked 4 Times in 3 Posts
JulieV is an unknown quantity at this point
The site is very bright and neonist type and most web browsers would shy away from those type of things. And its basically a web store and you will be up against the heavy weight of the in the business like Amazon and ebay. That would a tough niche to crack if you are the owner of the site.
JulieV is offline   Reply With Quote
Old 02-11-2009, 07:28 AM   PM User | #7
abduraooft
Supreme Master coder!

 
abduraooft's Avatar
 
Join Date: Mar 2007
Location: N/A
Posts: 14,678
Thanks: 158
Thanked 2,182 Times in 2,169 Posts
abduraooft is just really niceabduraooft is just really niceabduraooft is just really niceabduraooft is just really niceabduraooft is just really nice
Quote:
Originally Posted by x1ngxang View Post
@Millenia, thanks for the input. I'll look into those errors and redesign the layout to fix the table no-no.
While making a CSS design, you need to take care of making a semantic markup, and should never end up in any kind of divitis or something related.

Have a look at a good 2 column layout, http://www.bonrouge.com/2c-hf-fluid(r).php (you may set a fixed width to the #wrap, if required.)

Also, there is no need to use separate images for displaying your menu items. You could use a single image as background and then write the link text (as such in HTML) over them. Take a look at some CSS based menus at http://www.cssplay.co.uk/menus/
__________________
Quote:
The Dream is not what you see in sleep; Dream is the thing which doesn't let you sleep. --(Dr. APJ. Abdul Kalam)

Last edited by abduraooft; 02-11-2009 at 07:31 AM..
abduraooft is offline   Reply With Quote
Reply

Bookmarks

Jump To Top of Thread


Thread Tools
Rate This Thread
Rate This Thread:

Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

BB code is On
Smilies are On
[IMG] code is On
HTML code is Off

Forum Jump


All times are GMT +1. The time now is 11:18 AM.


Advertisement
Log in to turn off these ads.