Tek-Tips is the largest IT community on the Internet today!

Members share and learn making Tek-Tips Forums the best source of peer-reviewed technical information on the Internet!

  • Congratulations wOOdy-Soft on being selected by the Tek-Tips community for having the most helpful posts in the forums last week. Way to Go!

Multiple DropDown/Forms not working 1

Status
Not open for further replies.

KOVMoe

Programmer
Jun 30, 2004
34
US
As a simple minded programmer, I wrote this code for multiple drop down boxes. It works, for a while The user can go to 2 seperate locations then the code just stops working. Here is the code, Does anyone see what I have screwed up?

<head>
<script language="JavaScript">
function jumptolink1(what){
var selectedopt=document.weblinks1.weblinks1.options[what]
if (document.getElementById && selectedopt.getAttribute("target")=="newwin")
window.open(selectedopt.value)
else
window.location=selectedopt.value
}
</script>
<script language="JavaScript">
function jumptolink2(what){
var selectedopt=document.weblinks2.weblinks2.options[what]
if (document.getElementById && selectedopt.getAttribute("target")=="newwin")
window.open(selectedopt.value)
else
window.location=selectedopt.value
}
</script>
<script language="JavaScript">
function jumptolink3(what){
var selectedopt=document.weblinks3.weblinks3.options[what]
if (document.getElementById && selectedopt.getAttribute("target")=="newwin")
window.open(selectedopt.value)
else
window.location=selectedopt.value
}
</script>
<script language="JavaScript">
function jumptolink4(what){
var selectedopt=document.weblinks4.weblinks4.options[what]
if (document.getElementById && selectedopt.getAttribute("target")=="newwin")
window.open(selectedopt.value)
else
window.location=selectedopt.value
}
</script>

<script language="JavaScript">
function jumptolink5(what){
var selectedopt=document.weblinks5.weblinks5.options[what]
if (document.getElementById && selectedopt.getAttribute("target")=="newwin")
window.open(selectedopt.value)
else
window.location=selectedopt.value
}
</script>
<script language="JavaScript">
function jumptolink6(what){
var selectedopt=document.weblinks6.weblinks6.options[what]
if (document.getElementById && selectedopt.getAttribute("target")=="newwin")
window.open(selectedopt.value)
else
window.location=selectedopt.value
}
</script>
<script language="JavaScript">
function jumptolink7(what){
var selectedopt=document.weblinks7.weblinks7.options[what]
if (document.getElementById && selectedopt.getAttribute("target")=="newwin")
window.open(selectedopt.value)
else
window.location=selectedopt.value
}
</script>
<script language="JavaScript">
function jumptolink8(what){
var selectedopt=document.weblinks8.weblinks8.options[what]
if (document.getElementById && selectedopt.getAttribute("target")=="newwin")
window.open(selectedopt.value)
else
window.location=selectedopt.value
}
</script>

</head>

<table width="98%" align="center" cellpadding="0" cellspacing="0">

<tr>
<td>
<form name="weblinks1" method="post">
<select name="weblinks1" style="width: 300;">
<option>----WEB LINKS----</option>
<option value=" Tag Application</option>
<option value=" Ship Web Tag</option>
<option value=" <option value=" Master Look Up</option>
<option value=" Link</option>
<option value=" Phone Link</option>
<option value=" <option value=" <option value=" Down Order</option>
<option value=" Order</option>
<option value=" <option value=" Pick Ticket</option>
<option value="ftp://ftp.inside.ups.com/swr/EUD/Midsouth/automotive/content/forms/Delivery%20Schedule.xls">2004 Tire Delivery Schedule</option>
<option value="ftp://ftp.inside.ups.com/swr/EUD/Midsouth/automotive/content/districtcall/RC%20Piviot%202003.zip">RC Pivot 02-03(Zip File)2/04</option>
<option value="ftp://ftp.inside.ups.com/swr/EUD/Midsouth/automotive/content/districtcall/RC%20Piviot%202003.zip">Mast 4 Link</option>
<option value=" & Oil</option>
<option value=" File Down Load</option>
<option value=" & Labor</option>
<option value="../../automotive/content/EFA.htm">Enviromental</option>
<option value=" <option value=" <option value="../../automotive/content/COMPLIANCE.htm">Compliance Training</option>
<option value="../../automotive/content/TRAILER.htm">Trailer Registration</option>
<option value=" Request Forms</option>
</select>
<input type="button" value="Go" onClick="jumptolink1(document.weblinks1.weblinks1.selectedIndex)"><br>

</form>
</td>
</tr>
<tr>
<td>
<form name="weblinks2" method="post">
<select name="weblinks2" style="width: 300;">
<option>----HOT LIST----</option>
<option value="EMISSIONS.htm">GM 292 EMISSIONS</option>
<option value="ftp://ftp.inside.ups.com/swr/EUD/Midsouth/automotive/content/bestpractice/DMT%20GMC%204.3%20FUEL%20PUMPS.ppt">GM 4.3L FUEL PUMP DIAGNOSE</option>
<option value="ftp://ftp.inside.ups.com/swr/EUD/Midsouth/automotive/content/bestpractice/T444E%20HIGH%20PRESSURE%20LEAK.ppt">2001-2002 T444E HIGH PSI LEAK</option>
<option value="ftp://ftp.inside.ups.com/swr/EUD/Midsouth/automotive/content/bestpractice/T444E%20Wiring%20Harness%20Inspection.pdf">T444E WIRING CAMPAIGN</option>
<option value="ftp://ftp.inside.ups.com/swr/EUD/Midsouth/automotive/content/bestpractice/IHC%20T444E%20Wiring.ppt">IHC T444E WIRING PROBLEM</option>
<option value="ftp://ftp.inside.ups.com/swr/EUD/Midsouth/automotive/content/bestpractice/Tire%20Puncture%20Prevention.ppt">TIRE PUNCTURE PREVENTION</option>
<option value="ftp://ftp.inside.ups.com/swr/EUD/Midsouth/automotive/content/forms/ifta_license2004.pdf">IFTA DOCUMENTS</option>
<option value="ftp://ftp.inside.ups.com/swr/EUD/Midsouth/automotive/content/forms/2004-OH_Company_SSRS_Receipt.pdf">fuel PERMIT</option>
<option value="ftp://ftp.inside.ups.com/swr/EUD/Midsouth/automotive/content/Campaign%20Info/Camp%2048%209010%20Ford%20sycro%20start.doc">CAMP 48-9010 PROBLEM</option>
</select>
<input type="button" value="Go" onClick="jumptolink2(document.weblinks2.weblinks2.selectedIndex)"><br>

</form>
</td>
</tr>
<tr>
<td>
<form name="weblinks3" method="post">
<select name="weblinks3" style="width: 300;">
<option>----BEST PRACTICES----</option>
<option value="ftp://ftp.inside.ups.com/swr/EUD/Midsouth/automotive/content/bestpractice/DMT%20GMC%204.3%20FUEL%20PUMPS.ppt">GM 4.3L FUEL PUMP DIAGNOSE</option>
<option value="EMISSIONS.htm">GM 292 EMISSIONS</option>
<option value="ftp://ftp.inside.ups.com/swr/EUD/Midsouth/automotive/content/bestpractice/4.3%20elec.%20fuel%20pump%20install.doc">GM 4.3L FUEL PUMP INST</option>
<option value="ftp://ftp.inside.ups.com/swr/EUD/Midsouth/automotive/content/forms/caliper%20key.doc">GM CALIPER KEY PART #’S</option>
<option value="regioncall/4.3%20tensioner.htm">GM 4.3 TENSIONER PROBLEMS</option>
<option value="regioncall/Pulley%20installation.htm">GM 4.3 PULLEY INSTALL</option>
<option value="ftp://ftp.inside.ups.com/swr/EUD/Midsouth/automotive/content/bestpractice/4.3%20Water%20Pump%20Failure.ppt">GM 4.3 WATER PUMP PROBLEM</option>
<option value="ftp://ftp.inside.ups.com/swr/EUD/Midsouth/automotive/content/regioncall/Fan%20Belt%20and%20Pulley%20Maintenance%20%20292%20GMC.ppt">GMC 292 BELT MAINTENANCE</option>
<option value="ftp://ftp.inside.ups.com/swr/EUD/Midsouth/automotive/content/bestpractice/Tie%20rod%20endl.doc">GMC TIE ROD END RECALL</option>
<!-- <option value="ftp://ftp.inside.ups.com/swr/EUD/Midsouth/automotive/content/regioncall/Carburator.pps">GMC 292 CARB CHOKE FIX</option> -->
<option value="mechanics.htm">PRO LINK REFERENCE GUIDE</option>
<option value="mechanics.htm">TECH 1A REFERENCE GUIDE</option>
<option value="ftp://ftp.inside.ups.com/swr/EUD/Midsouth/automotive/content/bestpractice/FORD%20PUMP.doc">FORD FUEL PUMP RECAL</option>
<option value="ftp://ftp.inside.ups.com/swr/EUD/Midsouth/automotive/content/bestpractice/p32%20relay%20log.xls">P32 RELAY INSPECTION FORM</option>
<option value="ftp://ftp.inside.ups.com/swr/EUD/Midsouth/automotive/content/regioncall/ISB%20Fuel%20Port%20Installation%202.ppt">CUMMINS ISB FUEL PORT INST</option>
<option value="ftp://ftp.inside.ups.com/swr/EUD/Midsouth/automotive/content/bestpractice/CUMMINS%20ISB%20BELT..ppt">CUMMINS ISB BELT PROBLEMS</option>
<option value="ftp://ftp.inside.ups.com/swr/EUD/Midsouth/automotive/content/regioncall/T444e%20Glow%20Plug%20Relay.ppt">T444E GLOW PLUG RELAY</option>
<option value="ftp://ftp.inside.ups.com/swr/EUD/Midsouth/automotive/content/bestpractice/Oklahoma%20Automotivetest.ppt">T444E CAM SENSOR INSTALL</option>
<option value="ftp://ftp.inside.ups.com/swr/EUD/Midsouth/automotive/content/bestpractice/T444E%20OIL%20LINE.ppt">T444E HI-PSI OIL LINE RUPTURE</option>
<option value="mechanics.htm">T444E DIAGNOSTIC GUIDE</option>
<option value="ftp://ftp.inside.ups.com/swr/EUD/Midsouth/automotive/content/bestpractice/7.3%20accelerator%20cable.ppt">7.3 IHC ACCEL CABLE PROBLEM</option>
<option value="ftp://ftp.inside.ups.com/swr/EUD/Midsouth/automotive/content/bestpractice/P-47D%20Sprinter%20Electrical%20Problems1.ppt">P47 ECM WIRING PROBLEMS</option>
<option value="ftp://ftp.inside.ups.com/swr/EUD/Midsouth/automotive/content/regioncall/NP%20435%20SHIFT%20COVER%20PRESENTATION.ppt">NP435 SHIFT COVER</option>
<option value="mechanics.htm">435 465 REBUILD METHODS</option>
<option value="ftp://ftp.inside.ups.com/swr/EUD/Midsouth/automotive/content/regioncall/Spicer%20Detent.ppt">SPICER TRANS DETENT</option>
<option value="ftp://ftp.inside.ups.com/swr/EUD/Midsouth/automotive/content/regioncall/AUTOMOTIVE%20TIPS.doc">AUTOMOTIVE PCM TIPS</option>
<option value="ftp://ftp.inside.ups.com/swr/EUD/Midsouth/automotive/content/bestpractice/Cut%20Security%20Seals.ppt">CUT DOOR SEALS</option>
<option value="ftp://ftp.inside.ups.com/swr/EUD/Midsouth/automotive/content/bestpractice/Todco.doc">TODCO HINGE</option>
<option value="ftp://ftp.inside.ups.com/swr/EUD/Midsouth/automotive/content/bestpractice/Tire%20Puncture%20Prevention.ppt">TIRE PUNCTURE PREVENTION</option>
<option value="ftp://ftp.inside.ups.com/swr/EUD/Midsouth/automotive/content/forms/Belt%20List1.xls">BELT INVENTORY LISTING</option>
<option value="ftp://ftp.inside.ups.com/swr/EUD/Midsouth/automotive/content/bestpractice/RADIATOR%20PRESSURE%20CAPS%20P-C.doc">T-STAT REFERENCE</option>
<option value="ftp://ftp.inside.ups.com/swr/EUD/Midsouth/automotive/content/regioncall/DIAD%20Support%20Bumper.ppt">DVA FIX 2</option>
<option value="ftp://ftp.inside.ups.com/swr/EUD/Midsouth/automotive/content/bestpractice/4_3%20CABLE%20ADJ.doc">GM 4.3 ACCEL CABLE ADJ</option>
<option value="forms/BAT%20ID.JPG">BATTERY AGE ID CHART</option>
</select>
<input type="button" value="Go" onClick="jumptolink3(document.weblinks3.weblinks3.selectedIndex)"><br>

</form>
</td>
</tr>
<tr>
<td>
<form name="weblinks4" method="post">
<select name="weblinks4" style="width: 300;">
<option>----P47 SPRINTER INFORMATION----</option>
<option value="ftp://ftp.inside.ups.com/swr/EUD/Midsouth/automotive/content/forms/SPRINTER%20PARTS(2).xls">PARTS LIST</option>
<!-- <option value="ftp://ftp.inside.ups.com/swr/EUD/Midsouth/automotive/content/bpi/p47_pmi_interval.pdf">P47 PMI Intervals</option> -->
<option value="ftp://ftp.inside.ups.com/swr/EUD/Midsouth/automotive/content/Methods%20procedures/Sprinter%20Towing.doc">Towing document</option>
<option value="ftp://ftp.inside.ups.com/swr/EUD/Midsouth/automotive/content/bestpractice/P-47D%20Sprinter%20Electrical%20Problems1.ppt">P47 ECM WIRING PROBLEMS</option>
<option value="forms/UPS%20-%20Sprinter%20Fluids%20Lubricants.htm">SPRINT FLUID CHART</option>
<option value="forms/SPRINTER%20MANUAL.htm">SPRINT SERVICE MANUAL</option>
</select>
<input type="button" value="Go" onClick="jumptolink4(document.weblinks4.weblinks4.selectedIndex)"><br>
</form>
</td>
</tr>



Thank you,

[king]Moe-King of the Village Idiots.

"When in trouble,
when in doubt;
Run in circles-
SCREAM & SHOUT!!"
Burma Shave
 
Hi there...

I've optimized your page a bit, I hope it helps you out :)

1) You can make all that code into one function, as below:
Code:
<script language="JavaScript">
<!--

function jumpToLink(s) {
    var the_url = s.options[s.selectedIndex].value;
	document.location = the_url;
}

-->
</script>

2) You only need one form. Place the opening form tag outside of your <table> tag and close it outside your </table> tag. Get rid of all the other form tags.

3) The way you'd call this new function is like this (for each select box):

Code:
...
<input type="button" value="Go" onClick="jumpToLink(this.form.weblinks1);" /><br />
...
<input type="button" value="Go" onClick="jumpToLink(this.form.weblinks2);" /><br />
...
<input type="button" value="Go" onClick="jumpToLink(this.form.weblinks3);" /><br />
...
<input type="button" value="Go" onClick="jumpToLink(this.form.weblinks4);" /><br />

*cLFlaVA
----------------------------
Lois: "Peter, you're drunk!"
Peter: "I'm not drunk, I'm just exhausted from stayin' up all night drinking!
 
That did it, u DA MAN!!!!!!

Thank you,

[king]Moe-King of the Village Idiots.

"When in trouble,
when in doubt;
Run in circles-
SCREAM & SHOUT!!"
Burma Shave
 
Status
Not open for further replies.

Part and Inventory Search

Sponsor

Back
Top