Holiday apartments - simple CMSMS site

[designhut]

Hi again,

Just launched this one too, pretty basic variation on the default template, all comments and constructive criticisms invited.

www.lincombehouse.co.uk

I am getting an error on the page with IE7 due to an 'undefined' width, any hints appreciated!

I get a blank gap at the top of my header in FF2 that isn't in IE7...

Also, my custom copyright tag isn't showing up the '2007'. Never had that prob before...?

Is it just me but when I make changes in my Global Content Blocks, I cannot 'apply' them as it gives a smarty error without updating, so I have to 'submit' it. Sorry if this is a bug reported elsewhere...

Thx everyone!

[ironblaze94]

Really nice site ! 

Just one thing - http://validator.w3.org/check?uri=http%3A%2F%2Fwww.lincombehouse.co.uk%2F&charset=%28detect+automatically%29&doctype=Inline&group=0

[Dr.CSS]

If you give the body margin: 0 1em it will hit the top...

Looks real wide in IE6...

If you want a set size for both take out the script for IE and in your CSS...
change to this...

div#pagewrapper {
  border: 3px solid #58386a;
  margin: 0 auto;    /* this centers wrapper */
  width:700px;
  background-color:#fff;
  color: #58386a;
}

This for IE...  delete from template...


=b){nw=b+"px";}if(w



#pagewrapper {width:expression(P7_MinMaxW(700,700));}
#container

{height: 1%;}



Your line 27 column 1 - Warning: element not empty or not closed
you need to add a space then / before the last >...

country,beaches" />

[designhut]

Ironblaze94: Hi, thx so much, I'd be dangerous if I got it right!  All validated now I think.

Sven: Hi again, thx for your comments. The availability is a subscription package that the client already used, I just put the link in. I was looking at trying to use a Calendar module instead but I wimped out! Your suggestion to split the page up is spot on, I actually left it on purpose for the client to do when I give them their training session.

Mark: Hi oh helpful one! All errors appear to be fixed now, your assistance is invaluable and oh so appreciated! 
I'm still having the border issue, I tried your fix but it's not the margin around the body that I have a prob with, it appears to be in the header - a gap between the house image and the pagewrapper border, and I just can't see why...

Thx one and all! 

[Dr.CSS]

The gap at the top,between the page and the top of the browser window where you see background color purple, comes from the body tag having 1em margin if you use margin: 0px 1em it will hit the top...

The image in your header having a gap at the top, just white above it in Firefox, is from having margins around the h1 in your style sheet some where so...

div#header h1{margin:0px}

Should fix it... it doesn't show in IE6 with a gap...

[designhut]

Yes! That's the one! Thx Mark, you're a genius.