Combine two javascript

longcroft · #1 (**permalink**) Jun 3rd, 2008, 10:59

Hi I am having problems with my js. The problem is that it just wont work any ideas? Please find below the code from the head.

Code: Select all

<head>
<title></title>
<meta content="en-gb" http-equiv="Content-Language" />
<meta content="text/html; charset=utf-8" http-equiv="Content-Type" />
<link href="style.css" media="screen" rel="stylesheet" type="text/css" />
<link href="main%20nav/menu_style.css" rel="stylesheet" type="text/css" />
<link href="sdmenu/sdmenu.css" rel="stylesheet" type="text/css" />
<script src="mootools.js" type="text/javascript"></script>
<script src="slideshow.js" type="text/javascript"></script>
<script src="sdmenu/sdmenu.js" type="text/javascript">
  /***********************************************
  * Slashdot Menu script- By DimX
  * Submitted to Dynamic Drive DHTML code library: http://www.dynamicdrive.com
  * Visit Dynamic Drive at http://www.dynamicdrive.com/ for full source code
  ***********************************************/
 </script>
<script type="text/javascript">
 // <![CDATA[
 var myMenu;
 window.onload = function() {
  myMenu = new SDMenu("my_menu");
  myMenu.init();
 };
 // ]]>
 </script>
<script type="text/javascript">
{ myShow = new Slideshow('image', { type: 'fade', duration: [2000, 4000], images: ['header.jpg', 'School Site Photo May 06 Letterbox v2.jpg', 'hums2.JPG'], hu: 'images/' }); }
</script>
</head>

CloudedVision · #2 (**permalink**) Jun 3rd, 2008, 13:11

"Just won't work" isn't detailed enough explanation. What's it supposed to do? What's not working? (All of it?) Does it not work in all browsers? Does it cause any errors, and if so, can you post the errors? You gotta give us a lot more info to help you work through this.

Rob · #3 (**permalink**) Jun 3rd, 2008, 13:16

This is without doubt, almost certainly conflicting / incompatible scripts.

My guess is that you will need to start your bug-hunt by checking out al the window onload processes including any present in the body tag.

longcroft · #4 (**permalink**) Jun 3rd, 2008, 13:37

Think I have found the problem but just dont know how to solve this. There is two scripts,

Code: Select all

<script src="sdmenu/sdmenu.js" type="text/javascript">
  /***********************************************
  * Slashdot Menu script- By DimX
  * Submitted to Dynamic Drive DHTML code library: http://www.dynamicdrive.com
  * Visit Dynamic Drive at http://www.dynamicdrive.com/ for full source code
  ***********************************************/
 </script>
<script type="text/javascript">
 // <![CDATA[
 var myMenu;
 window.onload = function() {
  myMenu = new SDMenu("my_menu");
  myMenu.init();
 };
 // ]]>
 </script>

Then

Code: Select all

<script type="text/javascript">
 window.onload = function() { myShow = new Slideshow('image', { type: 'fade', duration: [2000, 4000], images: ['hums.JPG', 'hums1.JPG', 'hums2.JPG'], hu: 'images/' }); }
</script>

I think this is due to as Rob said windows on load function. Thanks for all your help

Rob · #5 (**permalink**) Jun 3rd, 2008, 13:44

Try this:--

Code: Select all

<script src="sdmenu/sdmenu.js" type="text/javascript">
  /***********************************************
  * Slashdot Menu script- By DimX
  * Submitted to Dynamic Drive DHTML code library: http://www.dynamicdrive.com
  * Visit Dynamic Drive at http://www.dynamicdrive.com/ for full source code
  ***********************************************/
 </script>
<script type="text/javascript">
 // <![CDATA[
 var myMenu;
 window.onload = function() {
  myMenu = new SDMenu("my_menu");
  myMenu.init();
myShow = new Slideshow('image', { type: 'fade', duration: [2000, 4000], images: ['hums.JPG', 'hums1.JPG', 'hums2.JPG'], hu: 'images/' });
 };
 // ]]>
 </script>

Delete this:-

Code: Select all

<script type="text/javascript">
 window.onload = function() { myShow = new Slideshow('image', { type: 'fade', duration: [2000, 4000], images: ['hums.JPG', 'hums1.JPG', 'hums2.JPG'], hu: 'images/' }); }
</script>

This *should* help.

Remember to use [code]your code here[/code] <--- this format when posting code

Hope this helps.

longcroft · #6 (**permalink**) Jun 3rd, 2008, 13:52

It worked. Its taken me all day and its taken you 5 mins to sort it.

Thanks for all your help. Its the firt time I have used this site and its been excellet I am designing a school website. Thanks Rob

Rob · #7 (**permalink**) Jun 3rd, 2008, 14:51

Most welcome

Be sure to tell all your web design friends

Feel free to post whenever you run into issues

Similar Threads
Thread	Thread Starter	Forum	Replies	Last Post
Date & Time [Combine]	doctypedeclaration	JavaScript Forum	3	Mar 12th, 2008 01:22
Coding help: Combine Insert into and If/else statement!	skuliaxe	PHP Forum	6	Jan 23rd, 2008 17:50
[SOLVED] combine 2 onlick events.	AdRock	JavaScript Forum	1	Oct 15th, 2007 06:03
what does + + mean in Javascript eg: var m = (A1+ +B4);	Andy K	JavaScript Forum	6	Dec 13th, 2006 23:57
How can I combine two columns into one column using SQL	gecastill	Databases	2	Jun 3rd, 2005 18:23

		Webforumz.com > Main Forums > Program Your Website > JavaScript Forum
Combine two javascript