DISPLAY is not a valid TAG ATTRIBUTE. it is just a style setting. ..setAttribute() is useless for setting DISPLAY STYLE. use William's suggested .style.display='none' / '' instead.
Azadi Saryev Sabai-dee.com http://www.sabai-dee.com/ Jeremy Bower wrote: > Correction: I just use insertRow() and deleteRow(), not innerHTML. I have > now tried this: > > eval("if (!document.all." + table_name + ".rows[1]) { my_new_row = > document.all." + table_name + ".insertRow(1); } else { my_new_row = > document.all." + table_name + ".rows[1]; }"); > eval("document.all." + table_name + ".rows[1].setAttribute('display','yes')"); > > for open and > eval("document.all." + table_name + ".rows[1].setAttribute('display','no')"); > > for close. No JS errors, but this does not close the row, nothing happens. > A getAttribute check after this shows display is changed, but nothing is > rendered to the screen. > > > > >> Have you tried >> .style.display = 'none' in javascript? Instead of the .innerHTML >> >> ---------------------------------- >> William Seiter >> ColdFusion Web Developer / Consultant >> http://william.seiter.com >> >> >> ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~| Adobe® ColdFusion® 8 software 8 is the most important and dramatic release to date Get the Free Trial http://ad.doubleclick.net/clk;203748912;27390454;j Archive: http://www.houseoffusion.com/groups/CF-Talk/message.cfm/messageid:309149 Subscription: http://www.houseoffusion.com/groups/CF-Talk/subscribe.cfm Unsubscribe: http://www.houseoffusion.com/cf_lists/unsubscribe.cfm?user=11502.10531.4