View Single Post
  #7 (permalink)  
Old April 23rd, 2004, 10:37 AM
ChrisScott ChrisScott is offline
Friend of Wrox
 
Join Date: Jul 2003
Location: , , United Kingdom.
Posts: 683
Thanks: 0
Thanked 1 Time in 1 Post
Default

Hi Luke,

A few things...

- you only need to reference the roll.js file once (there are two script tags linking to it)
- the body tag should only have one onload attribute
- your second onload attribute contains the following...

var SymTmpWinOpen = window.open;
window.open = SymWinOpen;
var SymTmpWinOpen = window.open;
window.open = SymWinOpen;
MM_preloadImages('img/home_nav_home_over.gif','img/home_nav_about_us_over.gif','img/home_nav_contact_us_over.gif','img/home_nav_selling_over.gif','img/home_nav_buying_over.gif',preload(),slideit();
window.open = SymTmpWinOpen;;
window.open = SymTmpWinOpen;

I'm not sure what you're trying to do with the SymTmpWinOpen = window.open and window.open = SymTmpWinOpen lines, but the preload images one should surely be something like...

MM_preloadImages('img/home_nav_home_over.gif','img/home_nav_about_us_over.gif','img/home_nav_contact_us_over.gif','img/home_nav_selling_over.gif','img/home_nav_buying_over.gif');

preload();
slideit();

If this is the case, there is no preload() function defined in your code & this will throw an error.

I think it would make it easier to move this code to a function and call that function from your onload tag.

Best regards,

Chris


Reply With Quote