Javascript is not working with IE

Hi All,

I am trying to hide html table row using javascript the code for that is below-

<%@ page language="java" contentType="text/html; charset=UTF-8" pageEncoding="UTF-8"%>
<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN" "http://www.w3.org/TR/html4/loose.dtd">
<html><head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
<title>Insert title here</title>

function RemoveRow()
         {
                 var x=document.getElementsByName("row1");
                 for(var i=0;i<x.length;i++)
                          {
                                 x[i].style.display = "none";

}    
         }
         function DisplayRow()
         {
                 var x=document.getElementsByName("row1");
                 for(var i=0;i<x.length;i++)
                          {
                         x[i].style.display  = "table-row";

}
         }

</script>
</head>
<body>
<form>

<table align="center">
                <tr name="row2"><td>--------- </td><td></td></tr>
                <tr name="row1"><td>Hi</td><td></td></tr>
                <tr name="row1"><td>These are</td><td></td></tr>
                <tr name="row1"><td>Contents</td><td></td></tr>
                <tr name="row2"><td>--------- </td><td></td></tr>
                <tr><td><input type="button" value="Show" onclick="DisplayRow()"></td><td><input type="button" value="Hide" onclick="RemoveRow()"></td></tr>
                
</table></form></body></html>

This code is working fine with all the browsers except IE.
Please help me.

Thank you.

First, please use code tags when posting code. Second, if this is the actual code there are many things wrong. There are no html or body tags, you are missing attributes for the form tag, and you are trying to declare a name attribute for td elements which is not allowed.

The first step is to create valid html. Then debug the JS. Try assigning ID's instead of names and then use getElementById. Come back here with the results after you've done those things.

Jk Robbins wrote:First, please use code tags when posting code. Second, if this is the actual code there are many things wrong. There are no html or body tags, you are missing attributes for the form tag, and you are trying to declare a name attribute for td elements which is not allowed.

The first step is to create valid html. Then debug the JS. Try assigning ID's instead of names and then use getElementById. Come back here with the results after you've done those things.

I make the changes,actually this is the simple code,in my actual code I am generating table rows dynamically,because of requirement I am using getElementByName.

Please repost the updated code. UseCodeTags, and make sure that code is properly indented.

Yes, I know you said the code was generated, so it's likely all wonky, bit take the time to reformat it for posting.

And as already pointed out -- if the HTML is invalid, all bets are off. Make suer that the markup is valid before trying to use script on it.