CODEWITHSUNDEEP
javascripthtml
Joe Engle
Joe Engle

Reputation: 197

Javascript InnerHTML text not changing

For some reason the ID= employer won't change to employerCont (which is 34.07. It just continues to say test1. Does anyone know why?

<div id="main" style="background-color:#F0F0F0; width:500px; height:500px;">&nbsp;</div>
<div id="content" style="background-color:gray; width:500px; height:500px;">
<p id="employer">test1</p>
<p id="employee">test2</p>
</div>

<script src="C:\Documents and Settings\zx08067\Desktop\HSA_RaphaelGraph\raphael.js"></script>
<script src="C:\Documents and Settings\zx08067\Desktop\HSA_RaphaelGraph\g.raphael.js"></script>
<script language="javascript">

function calculatePercentages (var EE, var ER) {
    var sum = EE + ER;
    var EE_perc = EE / sum;
    var ER_perc = ER / sum;

}

//employee and employer contributions
var employeeCont = 251.34;
var employerCont = 34.07;

document.getElementById("employer").innerHTML = employerCont;

calculatePercentages(employeeCont, employerCont);

var tee=[0,0,500,500, 
{type:"path", path:"M58.5,50, C58.5,43 51.5,43 51.5,50, C51.5,57 58.5,57 58.5,50", fill:"green", "stroke-width":"0"},
{type:"path", path:"M52,52 L58,52 L58,70 L55,78 L52,70 L52,52", fill:"green", "stroke-width":"0"}, 
{type:"path", path:"M57,49, C57,46 53,46 53,49 C53,52.5 57,52.5 57,50", fill:"white", "stroke-width":"0"}
];

document.getElementById("main") = Raphael(tee);

</script>

Upvotes: -1

Views: 1913

Answers (3)

Nick Chapman
Nick Chapman

Reputation: 4624

Similar to Cameron's answer, try calling once the document has finished loading. 

window.onload = function(){
    document.getElementById("employer").innerHTML = employerCont;
}

No jQuery required.

Upvotes: 1

AGDM
AGDM

Reputation: 104

Stripping away the non critical code to solve your specific question seemed to fix the issue. working code on JSBin

Upvotes: 0

akinuri
akinuri

Reputation: 12027

Mistype.

function calculatePercentages (var EE, var ER)

Remove vars.

function calculatePercentages (EE, ER)

FIDDLE

Upvotes: 5

Related Questions