Buoka Posted January 22, 2008 Share Posted January 22, 2008 I have tried to use the following with the javascript out in a .js file and with it in the <head> but in either case all I get is a call for debug.Can someone tell me where my code is wrong.The code calling the function is: <input class="accs" type="image" src="Images/SpAc_PumpSub.jpg" alt="Nutool Submersible Pump" title="Nutool Submersible Pump" value="SpAc01" onclick="SpAc('Accs/SpAc01.html').js" /> or <input class="accs" type="image" src="Images/SpAc_PumpSub.jpg" alt="Nutool Submersible Pump" title="Nutool Submersible Pump" value="SpAc01" onclick="SpAc('Accs/SpAc01.html')" /> The script in SpAc.js is: function SpAc(addr) {window.open(addr,'Spa Accessory','width=675,height=260');} The same code - unchanged completely - has been tried in the head thus: function SpAc(addr) {window.open(addr,'Spa Accessory','width=675,height=260'); } </head> Where does it all go pear-shaped?Thanks Link to comment Share on other sites More sharing options...
Buoka Posted January 22, 2008 Author Share Posted January 22, 2008 Just to clarify, the code for the javascript in the header reads: <script type="text/javascript"><!--function SpAc(addr, WinName) {window.open(addr, WinName,'width=675,height=260');}//--></script> Link to comment Share on other sites More sharing options...
Ingolme Posted January 22, 2008 Share Posted January 22, 2008 This code doesn't look right to me:<input class="accs" type="image" src="Images/SpAc_PumpSub.jpg" alt="Nutool Submersible Pump" title="Nutool Submersible Pump" value="SpAc01" onclick="SpAc('Accs/SpAc01.html').js" />And I see your function has two parameters:function SpAc(addr,WinName)and you're only calling it using one:SpAc('Accs/SpAc01.html')Other than that, you can remove the comments (<!-- -->) from the script tag, it really isn't much use. Link to comment Share on other sites More sharing options...
Buoka Posted January 22, 2008 Author Share Posted January 22, 2008 Thanks very much for help. Seems I have my thick head on today as...I still get no action and a debug request. Debugger narrative is "object required".Current code in html is: <input class="accs" type="image" src="Images/SpAc_PumpSub.jpg" alt="Nutool Submersible Pump" title="Nutool Submersible Pump" value="SpAc01" onclick="SpAc('Accs/SpAcPump.html', 'Nutool Submersible Pump').js" /> SpAc.js (capitals as shown) is: function SpAc(addr,WinName) {window.open(addr,WinName,'width=675,height=260');} Link to comment Share on other sites More sharing options...
Ingolme Posted January 22, 2008 Share Posted January 22, 2008 You problem is that you keep on adding .js where you shouldn't:<input class="accs" type="image" src="Images/SpAc_PumpSub.jpg" alt="Nutool Submersible Pump" title="Nutool Submersible Pump" value="SpAc01" onclick="SpAc('Accs/SpAcPump.html', 'Nutool Submersible Pump').js" />Take it away and it should look like this:<input class="accs" type="image" src="Images/SpAc_PumpSub.jpg" alt="Nutool Submersible Pump" title="Nutool Submersible Pump" value="SpAc01" onclick="SpAc('Accs/SpAcPump.html', 'Nutool Submersible Pump')" /> Link to comment Share on other sites More sharing options...
Buoka Posted January 22, 2008 Author Share Posted January 22, 2008 Thanks very much for help. Seems I have my thick head on today as...I still get no action and a debug request. Debugger narrative is "object required".Current code in html is:<input class="accs" type="image" src="Images/SpAc_PumpSub.jpg" alt="Nutool Submersible Pump" title="Nutool Submersible Pump" value="SpAc01" onclick="SpAc('Accs/SpAcPump.html', 'Nutool Submersible Pump')" /> SpAc.js (capitals as shown) is: function SpAc(addr,WinName) {window.open(addr,WinName,'width=675,height=260');} Changes made - thanks you very much again - still no working. What links this html to the .js file? How is that called? Link to comment Share on other sites More sharing options...
Ingolme Posted January 22, 2008 Share Posted January 22, 2008 If your javascript is in a separate page, then you have to put this in your <head> tags:<script src="file.js" type="text/javascript"></script>And inside the .js file you write the javascript without script tags, like this:function SpAc(addr,WinName) {window.open(addr,WinName,'width=675,height=260');} Link to comment Share on other sites More sharing options...
Buoka Posted January 23, 2008 Author Share Posted January 23, 2008 Thanks Ingol, but somehow it still won't work properly.Here is the code in SpAc.js function SpAc(addr,WinName) {window.open(addr,WinName,'width=675,height=260');} The code in the <head> section reads: <title>Systemwash UK Ltd - Spas, Saunas, Gazebos and Lifestyle Products - Howden, East Yorkshire</title> <script src="SpAc.js" type="text/javascript"></script> </head> and the html reads: <input class="accs" type="image" src="Images/SpAc_PumpSub.jpg" alt="Nutool Submersible Pump" title="Nutool Submersible Pump" value="SpAc01" onclick="SpAc('Accs/SpAcPump.html', 'Nutool Submersible Pump').js" /> All of which seem to be what I should have and how you confirm it should be. Why then does it all fall down? Link to comment Share on other sites More sharing options...
Buoka Posted January 23, 2008 Author Share Posted January 23, 2008 UPDATE - works in Firefox - but absolutely no action in IE (7). Ideas? Link to comment Share on other sites More sharing options...
Ingolme Posted January 23, 2008 Share Posted January 23, 2008 I'm still seeing an additional ".js" at the end of your function calling:<input class="accs" type="image" src="Images/SpAc_PumpSub.jpg" alt="Nutool Submersible Pump" title="Nutool Submersible Pump" value="SpAc01" onclick="SpAc('Accs/SpAcPump.html', 'Nutool Submersible Pump').js" />Take that away and the function should work just fine. Link to comment Share on other sites More sharing options...
Buoka Posted January 23, 2008 Author Share Posted January 23, 2008 Thanks so much Ingolme - I was so near yet so far lol. Ah well.Finally, this still fails in IE(7). I have narrowed it down (I think) to the spaces between words (or at least multiple words) in the WinName variable. Those with single word (e.g. 'Covershelf' and 'Traymate') work but not one other does (e.g. 'Tester Kit' and Cover Straps')What did I do wrong now? Link to comment Share on other sites More sharing options...
Ingolme Posted January 23, 2008 Share Posted January 23, 2008 Window names can't contain spaces. Anyways, Nobody can see the name of the window, it's just used as a reference for the program. I recommend using a simple name for the window, like "newWindow" or something like that. Link to comment Share on other sites More sharing options...
Recommended Posts
Archived
This topic is now archived and is closed to further replies.