radiator website site check please!!!

google@ildw.com
avatar
rank Enthusiast

Enthusiast


Posts: 60
Joined: 2006-10-17
Location: Houston, Texas Jeahhhh!

Hello,

here is the website:

http://www.mccartyradiators.com

Please, any comments are welcome. I want to do this website as professional as possible.
Thanks

Mauricio

http://www.ildw.com

bluesphee
bluesphee's picture
rank Regular

Regular


Posts: 15
Joined: 2007-08-05

Looks good!

I think that it looks great especially for the type of business it is. Nice work!

Under the "Radiator Sales and Repairs" the last menu item (5th down) is too long and doesn't look right (I'm using Opera though so maybe that's why)

I would consider using a different font for the main site title (McCarty Dr. Radiator Service). The drop shadow is ok, but is a little too obvious.

burlster
burlster's picture
rank Leader

Leader


Posts: 619
Joined: 2007-05-31
Location: Bournemouth

Height problem

"The best place to take a leak"... BRILLIANT, loved it! Anyway, where was I.

Oh yes, I found a small problem Sad

The menu items look like they might be fixed height causing the problem below.

I refer of course to the bottom item which has been cut off...

AttachmentSize
problemburlster.jpg40.69 KB

Have YOU said Hello yet?
The CSSCreator Hello Thread

google@ildw.com
google@ildw.com's picture
rank Enthusiast

Enthusiast


Posts: 60
Joined: 2006-10-17
Location: Houston, Texas Jeahhhh!

radiator

Yes, I changed the font in the title I didn't like it. It was the boring times new roman.

Well, now I changed and please take a look at the boxes, they don't render properly in ie7 why is that?
http://www.mccartyradiators.com

Mauricio

jeff_way
jeff_way's picture
rank Enthusiast

Enthusiast


Posts: 137
Joined: 2007-05-07
Location: Nashville

I think the site looks

I think the site looks really good. I've noticed that a lot people are using slightly transparent backgrounds for the drop down menus. I understand the style choice...but personally, I just think it makes the text harder to read.

The only other thing I noticed was that the text in the header at the top right is somewhat pixelated and blurry.

Other than that, great job! Smiling

-Jeffrey Way | <a href="http://www.detacheddesigns.com/blog" title="Detached Designs Blog">My Blog </a>

google@ildw.com
google@ildw.com's picture
rank Enthusiast

Enthusiast


Posts: 60
Joined: 2006-10-17
Location: Houston, Texas Jeahhhh!

watch your conditional comments

Hello,

I posted my question 1 week ago, I tried to find answers why the ie7 wasn't enclosing the floating containers correctly, I spend hours watching the code even I designed a new page for the problem. And suddenly the idea, I have a conditional comment for the floating containers hack for ie6, what happen was that I put in the conditional comment "for ie6 and later" that is why the code didn't work for ie7 who needs the exact same hack that ie6 needs.

Well, look at your conditional comments, the one that I use now is:

<!--[if IE]>
<link href="IE6HACK.css" type="text/css" media="all" rel="stylesheet" />
<![endif]-->

Cheers.

Mauricio
http://www.ildw.com