Click twice for function execution

[willat8]

Hi.

I'm having some problems with the execution of a javascript function trigged by clicking on a div. This only occurs if I have not clicked anywhere on the page since load: if it is the first thing I click then I start getting problems.

See the problem in action here: http://willat8.freehostia.com/

If you click on the startbutton in the bottom left hand corner before you do anything else you'll notice that nothing happens. If you click it again then the intended action occurs.

I'll include the code for those who can't access the website.


Javascript

document.onclick = startMenuActions
 
var startMenuOn = false
var clickedElement = "none"
 
function startOrbOnMouseOver() {
 
if (startMenuOn == false)
{
setBackground('startorb','url(http://willat8.freehostia.com/res/startorbon.png)')
}
}
 
 
function startOrbOnMouseOut() {
 
if (startMenuOn == false)
{
setBackground('startorb','url(http://willat8.freehostia.com/res/startorboff.png)')
}
}
 
 
function startMenuActions() {
 
document.getElementById("startmenu").onclick = function() {
clickedElement = 'startmenu'
}
document.getElementById("startorb").onclick = function() {
clickedElement = 'startorb'
}
 
if (clickedElement == "startorb" && startMenuOn == false)
{
setBackground('startorb','url(http://willat8.freehostia.com/res/startorbclick.png)');setDisplay('startmenu','block');startMenuOn = true
}
else if (clickedElement == "startorb" && startMenuOn == true)
{
setBackground('startorb','url(http://willat8.freehostia.com/res/startorbon.png)');setDisplay('startmenu','none');startMenuOn = false
}
 
if (clickedElement == "none")
{
setBackground('startorb','url(http://willat8.freehostia.com/res/startorboff.png)');setDisplay('startmenu','none');startMenuOn = false
}
 
clickedElement = "none"
}

Relevant HTML

<div id="startorb" onmouseover="startOrbOnMouseOver()" onmouseout="startOrbOnMouseOut()">
</div>

Any help as to how I could make this function execute on the first click would be much appreciated.

Will.

[Rsmastermind]

Dear Friend,

See what is happenong in your code on the first click the function is getting fired
i.e startMenuActions it is going in that parenthesis it is setting the function clickedElement = 'startmenu' on onclick of the div insted of firing the function in which clickedElement = 'startmenu' is there.

On second click the startMenuActions is getting fired then it is firing the function clickedElement = 'startmenu' .Thats why it is getting fired on the second click.

So you just directly check the condition that which is element is fired.Instead of assigning any function to it.

[willat8]

Dear Friend,

See what is happenong in your code on the first click the function is getting fired
i.e startMenuActions it is going in that parenthesis it is setting the function clickedElement = 'startmenu' on onclick of the div insted of firing the function in which clickedElement = 'startmenu' is there.

On second click the startMenuActions is getting fired then it is firing the function clickedElement = 'startmenu' .Thats why it is getting fired on the second click.

So you just directly check the condition that which is element is fired.Instead of assigning any function to it.

Thank you for your help Rsmastermind. Problem solved now :) (well, almost).