js+asp problem

[pelegk1]

i have this code
<tr id="trMostViewed">
<td bgcolor="<%=Application("RightBarTitlesColor")%>" HEIGHT="20" width="100%"><font color="White"><b>&nbsp; most viewed </b></font></td>
<td><img border="0" SRC="/images/corner.gif"></td>
</tr>
<tr>
<td>
<!-- #include File="Viewed.asp" -->
<br></td>
</tr>

in the viewed.asp file i have this code :
<%
if rstDoc.EOF=true then Response.Write "<SCRIPT> if (document.all['trMostViewed']) { document.all['trMostViewed'].style.visibility='none')} </SCRIPT>"
%>

but i am getting an error which says :
document.all.trMostViewed.style.visibility
is not an object!!!

why is that?how can i fix it?

[pelegk1]

helppppppppppp

[Gollum]

You'll probably want to use document.getElementById('trMostViewed') instead of document.all[...]

[pelegk1]

it steal dosent effect :

document.getElementById(""trMostViewed"").style.display='none';

i want it to make the trMostViewed invisible but it isnt working why?

[simpson97]

Try changing this line
['trMostViewed']) { document.all['trMostViewed'].style.visibility='none')}

to

['trMostViewed']) { document.all['trMostViewed'].style.visibility='hidden'}

Bob

[pelegk1]

but why didnt the display worked?

[simpson97]

First off 'none' doesnt work in IE 'hidden' does .
Also you had a round right bracket a end of line.
'trMostViewed']) { document.all['trMostViewed'].style.visibility='none')}

I assume you are browsing sniffing to tailor script to multiple browsers ie: layers for Netscape.

Bob

[pelegk1]

with the visibility!
and second i dwork only with IE and not with netscape !
and third whyshould my broswer sniff something?