Simple but elegant - review needed

[humair4]

hi ppl

I need a review on this site I tried to make it simple and elegant for a corporate company but there seems something wrong with it, I can't seem to figure it out could you guys help.

URL: http://www.edcpl.com/

any feedback would be appreciated.

thanks

 

[Bruce_KD]

At first glance, it does look very nice.
I'll go through the really nit-picky things though.

It seems the site is looking towards an older clientele. Not necessarily elderly, but definitely middle-aged. Users of this age are beginning to experience some eyesight difficulty. For many teenagers, the site is great. However, you may want to consider increasing the font size, as well as the Menu links size.

You seem to like small font, but the "®" on your slogan is abnormally large. Its still clear that it's there when its half the size, but it isn't threatening or bragging like I feel it is now.

The first thing I noticed when I looked was the left-most image. The foreground is blurry, and I remember blinking a few times to make sure it wasn't me. If it doesn't make people dizzy, it may at least be a slight distraction.

One of my biggest pet peeves about sites is that they will have wonderful CSS everywhere, but the buttons (such as the submit button in your contact form) and just default. They stick out as almost trashy.

Are you mysql_real_escape_string()-ing every bit of _POST and _GET data that comes in to your site. I did not try the contact form, but if I type...
Username: test
Password x'x
Into the login form, it gives me an odd error. Other queries actually tell me invalid information, but that one throws a "PHP Fatal error". If you've never heard of SQL injection, you need to wise up very quickly.

Also in the login form, why the hell is the password field an "<input type='text'>".
Isn't it wonderful that when I'm showing this awesome website to my boss, he gets to see my password when I go to login? No, it is not good. "<input type='password'>".

While the layout is good, I'm getting the distinct feeling that there are some holes in the server-side security of the site. Unless I'm wrong about this, you should really consider having an expert review everything under the hood.


Bruce

 

[aaronfalloon]

I'd say the font saying "Everything thing..." looks bad. Italicised Verdana just doesn't look good . The borders around the images don't do anything for the site, perhaps make them a little thinner? I usually like to go for a 1 pixel border, but that might just be me! Also I'd take the ".::" out of the page titles. Those characters serve no purpose, and might mess with your search engine rankings.

 

[weblord]

you know what that "Everything is Secure" tagline there arouses the curiosities of the hackers, it might be a risk for those corporate companies you are publishing their logos. There is no real secure server only those server that you don't publish on the internet most probably and we know it no one is exempted.