Closing a hidden div

You only need

document.getElementById('details').style.display='none';

Setting it to nothing simply removes the attribute.  You also don't need to set visibility. "display" is all you need.

I actually like the little code that toggles it rather than two separate links.  Here's how it works.  First, anytime you check for a CSS attribute using javascript, you must set one in the inline style, like I did below, using style="display: none".  Here we are toggling "display" so that's what we set.

We are going to use the ?: operator, which works like this.
variable = (condition) ? this if true : this if false

We are setting something (variable or attribute) depending on whether or not a condition is true.

If the display is already set to 'none', we are going to set it to 'block', and if not, we are setting it back to 'none'.

<a href="javascript: void(0)" onclick="document.getElementById('details').style.display=document.getElementById('details').style.display=='none'?'block':'none';">toggle</a>
<div id="details" class="hidden_details" style="display: none;">
    Some info.<br />
    Some info.<br />
    Some info.<br />
    Some info.<br />
    Some info.<br />
</div>

Select allOpen in new window

jamesbcox1980: How would this work with multiple divs I assume I would need to change the Element ID to the name of the new div like this?

&nbsp;&nbsp;<a href="javascript: void(0)" onclick="document.getElementById('details').style.display=document.getElementById('details').style.display=='none'?'block':'none';"><img src="../../../_resources/images/icons/archive.gif" alt="Post a Comment" style="border:none"/></a>


&nbsp;&nbsp;<a href="javascript: void(0)" onclick="document.getElementById('comment').style.display=document.getElementById('comment').style.display=='none'?'block':'none';"><img src="../../../_resources/images/icons/comment.gif" alt="Post a Comment" style="border:none"/></a>


&nbsp;&nbsp;<a href="javascript: void(0)" onclick="document.getElementById('delete').style.display=document.getElementById('delete').style.display=='none'?'block':'none';"><img src="_resources/images/icon/trash.jpg" alt="Delete Record" style="border:none"/></a>

Then use divs like this?
<Div id="details" class="hidden_details" style="display: none; border: solid;"><Div align="right" style="margin:10px;">&nbsp;&nbsp;  <a href="#" onclick="document.getElementById('details').style.display='none';return false;"><strong>Close:&nbsp;x</strong></a><BR /></Div> Details Div</Div>

<Div id="comment" class="hidden_comment" style="display: none; border: solid;"><Div align="right" style="margin:10px;">&nbsp;&nbsp;  <a href="#" onclick="document.getElementById('details').style.display='none';return false;"><strong>Close:&nbsp;x</strong></a><BR /></Div> Comment Div</Div>

<Div id="delete" class="hidden_delete" style="display: none; border: solid;"><Div align="right" style="margin:10px;">&nbsp;&nbsp;  <a href="#" onclick="document.getElementById('details').style.display='none';return false;"><strong>Close:&nbsp;x</strong></a><BR /></Div> Delete Div</Div>



At first this appears to work but after a couple clicks between the links at the first part I start getting a blank space almost like the div was there just not visible. Any ideas?

Yes that is correct, you would need to have a different ID for each link attached to a DIV.  You can create a function to make less typing, and that function can include an argument that contains the ID, they way dachusa showed it.  His function does precisely what mine does, but mine is less typing.

There's all kinds of stuff you could do using nextSibling.  You wouldn't even need to specify an id.  This really depends on the link being right before the DIV you want to hide, with no whitespace at all, so keep that in mind when you're trying to adjust it to your purposes.

<script type="text/javascript">
function toggleDiv(elem)
{
    elem.style.display=elem.style.display=='none'?'block':'none';
}
</script>
 
<table cellpadding="10" cellspacing="0" border="0">
    <tbody>
        <tr>
            <td>
<!-- I didn't move the DIV to the next line here so the the "nextSibling" would not be interpreted as a space or line break -->
                <a href="javascript: void(0)" onclick="toggleDiv(this.nextSibling)">toggle 1</a><div style="display: none;">
                    Some info.<br />
                    Some info.<br />
                    Some info.<br />
                    Some info.<br />
                    Some info.<br />
                    Some info.<br />
                </div>
            </td>
            <td>
                <a href="javascript: void(0)" onclick="toggleDiv(this.nextSibling)">toggle 1</a><div style="display: none;">
                    Some info.<br />
                    Some info.<br />
                    Some info.<br />
                    Some info.<br />
                    Some info.<br />
                    Some info.<br />
                </div>
            </td>
        </tr>
    </tbody>
</table>

Select allOpen in new window

I just saw that you are using "return false".  That may be what's causing the problem

Also, all your other Close X links are pointing to the "Details" div

Ok I think I got it going with what dachusa: had posted and I don't get the mysterious random blank space. I would like to thank you both for your help.