Have you tried
..style.display = 'none' in javascript? Instead of the .innerHTML
--
William Seiter
ColdFusion Web Developer / Consultant
http://william.seiter.com
Free Website Trade Publication Website Magazine
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. +
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
Sorry, should have noted I had tried:
eval(document.all. + table_name + .rows[1].style.display='yes');
and got a JS error Could not get the display property. Invalid argument.
DISPLAY is not a valid TAG ATTRIBUTE. it is just a style setting.
.setAttribute() is useless for setting DISPLAY STYLE.
YES is not a valid value for DISPLAY property. check the html/css reference.
use none (to hide), inline, block or [empty string] (to show)
an element.
there are other valid values, too, but support varies by browser and you
will unlikely use those for a TR tag display anyway.
also, you remember
I set up a text box and a text field to try to mimic what you are trying to do.
I was able to come up with this, somewhat rudimentary, concept below. I hope
that it might help you to debug.
var submitRow = function () {
var newmssg = document.getElementById('message').value;
6 matches
Mail list logo
