PDA

View Full Version : Linking to external javascript file


Jules1777
07-10-2009, 02:14 PM
Hi

I have designed a website www.mentalhealthstrategies.co.uk and my javascript at the minute is written in the head section of every page. I want to put this javascript code into an external file called mhs.js, I have done this but am not sure how to link each page to the external file.

I have tried the following code in the head section of every page but this doesn't seem to work
<script language="javascript" src="mhs.js"></script>

What am I doing wrong?

Please help
Thanks in advance
Julie :confused:

Corrosive
07-10-2009, 02:27 PM
Hi

I have designed a website www.mentalhealthstrategies.co.uk and my javascript at the minute is written in the head section of every page. I want to put this javascript code into an external file called mhs.js, I have done this but am not sure how to link each page to the external file.

I have tried the following code in the head section of every page but this doesn't seem to work
<script language="javascript" src="mhs.js"></script>

What am I doing wrong?

Please help
Thanks in advance
Julie :confused:

Just a suggestion but try...

<script type="text/javascript" src="mhs.js"></script>

and check the path to the script is correct as well.

Jules1777
07-10-2009, 02:33 PM
Thanks Corrosive, have tried this and it didn't work. Any more suggestions?

Julie

DWcourse
07-10-2009, 02:47 PM
In the .js file don't include the opening
<script type="text/JavaScript">
<!--

or the closing

//-->
</script>

Jules1777
07-10-2009, 02:49 PM
Below is my .js file

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_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_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];}
}

Is this correct?

Julie

Corrosive
07-10-2009, 02:49 PM
In the .js file don't include the opening
<script type="text/JavaScript">
<!--

or the closing

//-->
</script>

Good shout that man :)

jaymo
07-12-2009, 04:29 PM
I would use css for rollovers instead of javascript.

http://www.webcredible.co.uk/user-friendly-resources/css/rollover-buttons.shtml