dynamicaly creating radio button does not work in IE

sam wrote :

Hi all,
I am dynamically creating a table rows and inerting radio buttons which
are also dynamically created. Everything works fine in Firefox as
expected. But I am not able to select radio buttons in IE.
You mean check radio buttons.

It does not even throw any errors.
When you seek help, it is always best to provide an url along with the
message text of the javascript error being reported.
I have searched over the net but could not find
anyhelp. Hope some experts here could help me.
Here is part my code that dynamically generates the radio buttons, I
cannot paste the entire code as it is very huge.

var n = 0;
var row = table.insertRow (table.rows.len gth);
You may want to consider -1 instead of requiring the browser to
calculate the table.rows.leng th. What you do though (using
table.rows.leng th) is more self-explanatory, intuitive, helps code
maintenance.

var objTRow = table.insertRow (-1);
"If index is -1 or equal to the number of rows, the new row is appended."
http://www.w3.org/TR/DOM-Level-2-HTM...ml#ID-39872903

var cell = row.insertCell( row.cells.lengt h);
var objTCell = row.insertCell(-1);

"If index is -1 or equal to the number of cells, the new cell is appended."
http://www.w3.org/TR/DOM-Level-2-HTM...ml#ID-68927016

cell = row.insertCell( row.cells.lengt h);
You have twice the same assignment here.
var radio = document.create Element('input' );
radio.setAttrib ute('type', 'radio');
MSIE 6 is DOM 1 (HTML interface) compliant but is not DOM 2 compliant.

http://www.w3.org/TR/2000/WD-DOM-Lev...tml#ID-6043025

http://www.w3.org/TR/DOM-Level-2-HTM...tml#ID-6043025

Type was read-only in DOM 1; this was changed in DOM 2.
radio.setAttrib ute('name', 'test'+n);
One other thing: you should prefer DOM 2 HTML interface methods instead
of using setAttribute since setAttribute is less widely supported in
browsers or is less buggy.
radio.setAttrib ute('value', 'test123');
radio.setAttrib ute('onclick', "this.form['test"+n+"'].disabled =
true;");
This is likely not to succeed. Event attributes are not set that way.
Instead:

radio.onclick = new Function("evt", "this.form['test"+n+"'].disabled =
true;")
cell.appendChil d(radio);
cell = row.insertCell( row.cells.lengt h);

Any help really aprreciated.

thanks,
Sam

Dynamically create dynamically radio buttons
http://www.gtalbot.org/DHTMLSection/...ioButtons.html

Please read my copyright notice:
http://www.gtalbot.org/Varia/CopyrightNotice.html

MSIE 6 bug: Dynamically inserted radio button is not checkable
http://www.gtalbot.org/BrowserBugsSe...dioButton.html

which I have reported already to MSIE 7 dev. team.

Gérard
--
remove blah to email me

sam said on 15/03/2006 10:26 AM AEST:

Hi all,
I am dynamically creating a table rows and inerting radio buttons which
are also dynamically created. Everything works fine in Firefox as
expected. But I am not able to select radio buttons in IE. It does not
You can have a radio button in the source HTML that is hidden (say set
its display attribute to 'none') and then clone it whenever you want to
add other.

even throw any errors. I have searched over the net but could not find
anyhelp. Hope some experts here could help me.
Here is part my code that dynamically generates the radio buttons, I
cannot paste the entire code as it is very huge.

var n = 0;
var row = table.insertRow (table.rows.len gth);
var cell = row.insertCell( row.cells.lengt h);

cell = row.insertCell( row.cells.lengt h);
var radio = document.create Element('input' );
Somewhere in your HTML source put:

<input type="radio" id="baseRadioBu tton" style="display: none;">

Then (with appropriate feature testing) clone it as required:

var radio =
document.getEle mentById('baseR adioButton').cl oneNode(false);

radio.style.dis play = '';

radio.setAttrib ute('type', 'radio');
radio.setAttrib ute('name', 'test'+n);
radio.setAttrib ute('value', 'test123');
radio.setAttrib ute('onclick', "this.form['test"+n+"'].disabled =
true;");
And then (remember to change the ID):

radio.id = 'test' + n;
radio.name = 'test' + n;
radio.value = 'test123';
radio.onclick = new Function(
'this.form["test' + n + '"].disabled = true;');

cell.appendChil d(radio);
cell = row.insertCell( row.cells.lengt h);

Any help really aprreciated.

thanks,
Sam

--
Rob

For IE, do it by means of property, setAtrribute() is correct,
however IE has issues with setAttribute() :/ , use instead,
radio.type=..; radio.onclick=. ..;
Danny

> "sam" <za*****@gmail. com> wrote:

news:11******** **************@ v46g2000cwv.goo glegroups.com.. ..

Hi all,
I am dynamically creating a table rows and inerting radio buttons
which are also dynamically created. Everything works fine in
Firefox as expected. But I am not able to select radio buttons in
IE. It does not even throw any errors. I have searched over the net
but could not find anyhelp. Hope some experts here could help me.
Here is part my code that dynamically generates the radio buttons, I
cannot paste the entire code as it is very huge.

[snip]

<!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=windows-1252">
<!--[if IE]>
<script type="text/javascript">
var IE=1;
</script>
<![endif]-->
<script type="text/javascript">
var n=0;
function addRadio(){
var i=0;
while(i<5){
if(typeof(IE)== 'number'){
j=document.crea teElement('<inp ut name=" ">');
}
else {
j=document.crea teElement('inpu t');
}
j.name='willy';
j.type='radio';
j.value=++n;
document.body.a ppendChild(j);
label=document. createElement(' label');
j.id=label.html For='willy'+n;
label.appendChi ld(document.cre ateTextNode('wi lly'+n));
document.body.a ppendChild(labe l);
document.body.a ppendChild(docu ment.createElem ent('br'));
i++;
}
}
</script>
<title></title>
</head>
<body>
<p><button onclick="addRad io()">Add Radio button</button></p>
</body>
</html>

--
BootNic Wednesday, March 15, 2006 2:59 AM

"I suppose they are vicious rascals, but it scarcely matters what
they are. I'm after what they know."
*Gibson-Sterling, The Difference Engine*

Thanks to one and all who responded .. All the answers really helped me
inprove my code and now it works in IE as well..

Thanks,
Sam