coool Posted August 12, 2007 Share Posted August 12, 2007 what do you think of this code? - it's not working what's wrong with it ! <script type="text/javascript"> function Show(e) { if(document.getElementById(e).style.display == 'none') { document.getElementById(e).style.display = 'block'; } } </script> <tr id='criteria1'> <?echo CriteriaRow();?> //this function have some form elements/html similar to the picture in one of my posts </tr> <input type="button" value="Display Criteria" onClick="Show('criteria1');"> #criteria1 { display: none; } when i click on the button.. nothing appear ! Quote Link to comment Share on other sites More sharing options...
gurroa Posted August 12, 2007 Share Posted August 12, 2007 Cause css style doesn't get into the element style definition. So the default value of style.display is empty string. function Show(e) { document.getElementById(e).style.display = document.getElementById(e).style.display != 'block' ? 'block' : 'none'; delete e; } Quote Link to comment Share on other sites More sharing options...
nogray Posted August 14, 2007 Share Posted August 14, 2007 instead of having the display in the style sheet declaration, have it in an inline style <tr style="display:none;" id="....."> .... Otherwise the if statment won't work (the style.display will be blank) Quote Link to comment Share on other sites More sharing options...
Recommended Posts
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.