ansaurus

Question

Answer 1

+6 A:

There are several issues:

Class names (and IDs) are not allowed to start with a digit.
You have to pass a class to getElementsByClassName().
You have to iterate of the result set.

Example (untested):

<script type="text/javascript">
function hideTd(className){
    var elements = document.getElementsByClassName(className);
    for(var i = 0, length = elements.length; i < length; i++) {
       if( elements[i].textContent == ''){
          elements[i].style.display = 'none';
       } 
    }

  }
</script>
</head>
<body onload="hideTd('td');">
<table border="1">
  <tr>
    <td class="td">not empty</td>
  </tr>
  <tr>
    <td class="td"></td>
  </tr>
  <tr>
    <td class="td"></td>
  </tr>
</table>
</body>

Note that getElementsByClassName() is not available up to and including IE8.

Update:

Alternatively you can give the table an ID and use:

var elements = document.getElementById('tableID').getElementsByTagName('td');

to get all td elements.

To hide the parent row, use the parentNode property of the element:

elements[i].parentNode.style.display = "none";

Felix Kling 2010-07-28 01:52:08

it's working now! but if getElementsByClassName() is not available in IE, is there an alternative way to hide an empty td without referencing each one? Also, is there a way to hide the whole tr if the td is empty? Thanks

new star 2010-07-28 02:04:29

@new star: Please see my updated answer.

Felix Kling 2010-07-28 02:13:33

thanks for the alternative!

new star 2010-07-28 02:28:40

Answer 2

+1 A:

If you want to do it by ClassName you could do:

<script type="text/javascript">
function hideTd(className){
    var elements;

    if (document.getElementsByClassName)
    {
        elements = document.getElementsByClassName(className);
    }
    else
    {
        var elArray = [];
        var tmp = document.getElementsByTagName(elements);  
        var regex = new RegExp("(^|\\s)" + className+ "(\\s|$)");
        for ( var i = 0; i < tmp.length; i++ ) {

            if ( regex.test(tmp[i].className) ) {
                elArray.push(tmp[i]);
            }
        }

        elements = elArray;
    }

    for(var i = 0, i < elements.length; i++) {
       if( elements[i].textContent == ''){
          elements[i].style.display = 'none';
       } 
    }

  }
</script>

2010-07-28 11:19:01

ansaurus

tags:

views:

answers:

getElementsByClassName not working

related questions