var quantityBox = document.getElementById("quantityBox");

if (quantityBox.style.display == "block") {
// function here
}

I can't seem able to get this to work. Is the code above correct?

depends,

do you specifically set the display value first somewhere?

Is the code below the HTML element with an ID of "quantityBox" (preferrably at the end of the page), or inside of an onload event? Or is it just sitting in the head, with nothing triggering it?

Originally Posted by chrishirst

depends,

do you specifically set the display value first somewhere?

Yes I have. The display is set to either block or none depending on what page they are viewing

Originally Posted by wayfarer07

Is the code below the HTML element with an ID of "quantityBox" (preferrably at the end of the page), or inside of an onload event? Or is it just sitting in the head, with nothing triggering it?

quantityBox is an Id set in the body and inside the html

Originally Posted by jason_bristol

quantityBox is an Id set in the body and inside the html

That's not what I asked. If your script isn't in a place where it will have access to the HTML, it won't matter.

The reason I bring this up (you haven't given us much of a clue since we can't see the whole page), is that many people who are new to JavaScript don't realize that scripts will have absolutely zero access to the HTML unless:

1. The script is inside of or triggered by an onload event.
2. The script appears somewhere below the HTML that it needs access to.

That said, there isn't anything actually wrong with the code you showed us.