Javascript works in Firefox, not in IE

saxman13ga posted this at 22:22 — 22nd April 2008.

Joined: May 2006

I'm having problems with a javascript, and I think its something simple that I'm missing. I tried researching and messing with it, but I could not find anything.

Basically, its a drop down menu to choose times, but they are only allowed to choose a time between 8am and 8pm. When you click on the hour, it changes the AM/PM dropdown to what it could be and disables it. It will be displayed more than once on a page, so it sends a variable called num to differentiate between different times. I tested it in firefox, and it works fine but in IE(6 & 7) it does not do anything.

Thank you for your help!

&lt;script type="text/javascript"&gt;
//time fix
function fixtimes(num) {
	var x = document.getElementById("time"+num+"a");//        am/pm
	var t = document.getElementById("time"+num+"h").value;//  hour
	if (t == 8 ) { //set to am and enable
		x.selectedIndex= 0; 
		x.disabled =false;
	} else if (t >= 9 && t <= 11) {//set to am and disable
		x.selectedIndex = 0; 
		x.disabled =true;
	} else if (t == 12 || (t >= 1 && t <= 7)){ //set to pm
		x.selectedIndex = 1; 
		x.disabled =true;
	} else { //should not happen
		x.disabled =false;
	}
}
&lt;/script&gt;


<p>Time 1:</p>
<select name="time1h" id="time1h" onChange="return fixtimes('1')">
	<option selected></option>
	<option >1</option>
	<option >2</option>
	<option >3</option>
	<option >4</option>
	<option >5</option>
	<option >6</option>
	<option >7</option>
	<option >8</option>
	<option >9</option>
	<option >10</option>
	<option >11</option>
	<option >12</option>
</select>
<select name="time1a" id="time1a" >
	<option selected>a.m.</option>
	<option >p.m.</option>
</select>

JeevesBond posted this at 18:17 — 27th April 2008.

He has: 3,956 posts

Joined: Jun 2002

You need to add a value attribute to all those options, for example:

      <option value="1">1</option>
      <option value="2">2</option>
      <option value="3">3</option>

Am attaching a working version of your code. The problem is that Internet Explorer won't use the text inside an <option></option> tag, the option="x" has to be included. Most other browsers work correctly, including Firefox and Opera.

a Padded Cell our articles site!