Hey all I'm a hack of a web designer and need help. I'm using DW templates for the first time and not sure if that's the problem or my css. (I know there's some goofy code for a javascript butterfly in the code...the client wants it. I removed it and still had the same problem.)

This page renders fine in Firefox, but doesn't in Safari or IE8.

http://www.renaissancemansion.com/te...s/aboutus.html


Thanks for any help!

<script type="text/javascript">
<!--
function MM_swapImgRestore() { //v3.0
  var i,x,a=document.MM_sr; for(i=0;a&&i<a.length&&(x=a[i])&&x.oSrc;i++) x.src=x.oSrc;
}
function MM_preloadImages() { //v3.0
  var d=document; if(d.images){ if(!d.MM_p) d.MM_p=new Array();
    var i,j=d.MM_p.length,a=MM_preloadImages.arguments; for(i=0; i<a.length; i++)
    if (a[i].indexOf("#")!=0){ d.MM_p[j]=new Image; d.MM_p[j++].src=a[i];}}
}

function MM_findObj(n, d) { //v4.01
  var p,i,x;  if(!d) d=document; if((p=n.indexOf("?"))>0&&parent.frames.length) {
    d=parent.frames[n.substring(p+1)].document; n=n.substring(0,p);}
  if(!(x=d[n])&&d.all) x=d.all[n]; for (i=0;!x&&i<d.forms.length;i++) x=d.forms[i][n];
  for(i=0;!x&&d.layers&&i<d.layers.length;i++) x=MM_findObj(n,d.layers[i].document);
  if(!x && d.getElementById) x=d.getElementById(n); return x;
}

function MM_swapImage() { //v3.0
  var i,j=0,x,a=MM_swapImage.arguments; document.MM_sr=new Array; for(i=0;i<(a.length-2);i+=3)
   if ((x=MM_findObj(a[i]))!=null){document.MM_sr[j++]=x; if(!x.oSrc) x.oSrc=x.src; x.src=a[i+2];}
}
</script>

You didn't close a comment tag. Most browsers will not see anything beyond.

I won't go into the other errors in your markup as they aren't causing major problems, YET.

Originally Posted by tinimic

<script type="text/javascript">
<!--
function MM_swapImgRestore() { //v3.0
  var i,x,a=document.MM_sr; for(i=0;a&&i<a.length&&(x=a[i])&&x.oSrc;i++) x.src=x.oSrc;
}
function MM_preloadImages() { //v3.0
  var d=document; if(d.images){ if(!d.MM_p) d.MM_p=new Array();
    var i,j=d.MM_p.length,a=MM_preloadImages.arguments; for(i=0; i<a.length; i++)
    if (a[i].indexOf("#")!=0){ d.MM_p[j]=new Image; d.MM_p[j++].src=a[i];}}
}

function MM_findObj(n, d) { //v4.01
  var p,i,x;  if(!d) d=document; if((p=n.indexOf("?"))>0&&parent.frames.length) {
    d=parent.frames[n.substring(p+1)].document; n=n.substring(0,p);}
  if(!(x=d[n])&&d.all) x=d.all[n]; for (i=0;!x&&i<d.forms.length;i++) x=d.forms[i][n];
  for(i=0;!x&&d.layers&&i<d.layers.length;i++) x=MM_findObj(n,d.layers[i].document);
  if(!x && d.getElementById) x=d.getElementById(n); return x;
}

function MM_swapImage() { //v3.0
  var i,j=0,x,a=MM_swapImage.arguments; document.MM_sr=new Array; for(i=0;i<(a.length-2);i+=3)
   if ((x=MM_findObj(a[i]))!=null){document.MM_sr[j++]=x; if(!x.oSrc) x.oSrc=x.src; x.src=a[i+2];}
}
</script>

You didn't close a comment tag. Most browsers will not see anything beyond.

I won't go into the other errors in your markup as they aren't causing major problems, YET.

Thanks that was it! Somehow that must have gotten clipped off. Did you see something else that was bad news??

Did you see something else that was bad

Your code is a mess of inline styles and deprecated code, for one thing.
<font> tags are a thing of the past.
You're using that messy DW garbage for image rollovers when they are completely unnecessary, it can be done with just css.

Invalid id names on images - they can't have spaces.

Multiple unnecessary spans; spacer gifs?? ugly stuff.

Like I said I'm a hack. I'm in the process of updating this from seven years ago. Thanks for looking and your tips.

D

Your code is a mess of inline styles and deprecated code, for one thing.
<font> tags are a thing of the past.
You're using that messy DW garbage for image rollovers when they are completely unnecessary, it can be done with just css.

Invalid id names on images - they can't have spaces.

Multiple unnecessary spans; spacer gifs?? ugly stuff.

Thank you LadynRed. I just started learning HTML and CSS a couple month's ago in order to build my first website. I had to suspend the project when I found that my initial markup was garbage (I was using DW and thought I was learning something by watching the code generated). I am learning much more by troubleshooting problems posted in the forums, but I still can't articulate the results of my efforts.

I am continually amazed at how well the browsers do parse some of the tag soup that I am seeing. Thank you again for a critique in concise, descriptive, and understandable format. I'll learn.

@dfunk44: many problems in markup can be found by using the W3C validation service at http://"http://validator.w3.org/"]ht...idator.w3.org/.

Thanks for everyone's comments. Here's the cleaned up version. The spacer gifs are staying until I figure out how to get the height 100%.

It's getting there....
http://www.renaissancemansion.com/test/aboutus.html

See our stickies on how to achieve the 100% height - but even with those solutions, there may be problems.