Welcome to Experts Exchange

Add your voice to the tech community where 5M+ people, just like you, are talking about what matters.

  • Help others & share knowledge
  • Earn cash & points
  • Learn & ask questions

Javascript in IE7

Posted on 2013-02-05
Last Modified: 2013-02-05
I am having a problem with some javascript. It seems to work on Chrome and IE9 but not in IE7. There are several radio options in the code below, one selects the product type (Battery, rechargeable or wired) and another selects the payment option (1 or 3 year package). The selections of these two controls should then set the price, but IE7 gets the result wrong in every case. It even gets a different answer for the same selections depending what happened before.

Clicking the sub, vat or total boxes, the values change to the correct value. Any ideas please?

<!DOCTYPE html>
<html lang="en">
  <meta charset="utf-8">
  <title>some title</title>
<!--  <link rel="stylesheet" media="screen" href="basic.css"> -->
  <link rel="stylesheet" media="screen", media="screen and (max-width:800px)" href="basic.css">
  <link rel="stylesheet" media="screen and (min-width:801px) and (max-width:1000px)" href="medium.css">
  <link rel="stylesheet" media="screen and (min-width:1001px)" href="wide.css">
  <!--[if (IE 8)|(IE 7)]>
    <link rel="stylesheet" media="screen" href="medium.css">
  <script type="text/javascript">
  function delAddr()
	if (document.getElementById("alt").checked)

  function calc()
    price = 0;
	if (document.getElementById("batt").checked)
		if (document.getElementById("oneyr").checked)
	else if (document.getElementById("rech").checked)
		if (document.getElementById("oneyr").checked)
	else/* if (document.getElementById("wire").checked)*/
		if (document.getElementById("oneyr").checked)
	sub = price;
	vat = sub * 0.2;
	price = price + vat;
	document.getElementById("subTot").value = sub.toFixed(2);
	document.getElementById("vat").value = vat.toFixed(2);
	document.getElementById("cost").value = price.toFixed(2);
<body id="buy">
  <div id="wrapper">
	<img id="top" src="tophome.jpg" alt="">
	<div id="wrapper2">
	  <div id="main">
		  <p>Some text</p>
		  <div id="buybox">
			<input type="hidden" name="instId"  value="210107">
			<input type="hidden" name="cartId" value="WS-01">
			<input type="hidden" name="currency" value="GBP">
			<input type="hidden" name="hideCurrency">
			<input type="hidden" name="desc" value="" id="desc">
			<input type="hidden" name="testMode" value="0">
			<input type="hidden" id="withDelivery" name="withDelivery" value="false">
			<p>Buy button takes you to the Worldpay Secure Server to enter your details safely</p>
			<p><label for="batt">Battery</label><input type="radio" id="batt" name="moption" value=0 checked="checked" onchange="calc()"></p>
			<p><label for="rech">Rechargeable</label><input type="radio" id="rech" name="moption" value=1 onchange="calc()"></p>
			<p><label for="wire">Wired</label><input type="radio" id="wire" name="moption" value=2 onchange="calc()"></p>

			<p><em>Payment option:</em></p>
			<p><label for="oneyr">1 year</label><input type="radio" id="oneyr" name="payoption" value=0 checked="checked" onchange="calc()"></p>
			<p><label for="threeyr">3 year</label><input type="radio" id="threeyr" name="payoption" value=1 onchange="calc()"></p>

			<p><em>Deliver to:</em></p>
			<p><label for="card">Card Address</label><input type="radio" id="card" name="deloption" value=0 checked="checked" onchange="delAddr()"></p>
			<p><label for="alt">Alternate Address</label><input type="radio" id="alt" name="deloption" value=1 onchange="delAddr()"></p>

			<p><label>Sub Total &pound</label><input type="text" id="subTot" name="subTot" readonly="readonly" value=0></p>
			<p><label>Vat &pound</label><input type="text" id="vat" name="vat" readonly="readonly" value=0></p>
			<p><label>Total &pound</label><input type="text" id="cost" name="cost" readonly="readonly" value=0></p>
		    <div class="clear"></div>
			<input type="submit" value="Buy Now" name="SUBMIT">

Open in new window

Question by:ncw
LVL 13

Accepted Solution

stergium earned 500 total points
ID: 38854719
there is a bug in IE7 about onchange event
you can use onclick or onblur event.
hope that helps
LVL 82

Expert Comment

ID: 38854738
try to clear the ie7 browser cache and reopen the page

Author Closing Comment

ID: 38855665
onclick worked, thanks.

Featured Post

Announcing the Most Valuable Experts of 2016

MVEs are more concerned with the satisfaction of those they help than with the considerable points they can earn. They are the types of people you feel privileged to call colleagues. Join us in honoring this amazing group of Experts.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

Suggested Solutions

Title # Comments Views Activity
Select case on click 3 19
Make Session back to empty 9 30
I thought I had multiple sliders on one page working properly . . . I don't. 10 16
Javascript 2 20
The task A number given should be formatted for easy reading by separating digits into triads. Format must be made inline via JavaScript, i.e., frameworks / functions are not welcome. So let’s take a number like this “12345678.91¿ and format i…
Today, the web development industry is booming, and many people consider it to be their vocation. The question you may be asking yourself is – how do I become a web developer?
The viewer will learn the basics of jQuery, including how to invoke it on a web page. Reference your jQuery libraries: (CODE) Include your new external js/jQuery file: (CODE) Write your first lines of code to setup your site for jQuery.: (CODE)
The viewer will learn the basics of jQuery including how to code hide show and toggles. Reference your jQuery libraries: (CODE) Include your new external js/jQuery file: (CODE) Write your first lines of code to setup your site for jQuery…

808 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question