Hi,
I recently tried using the HTMLRadioButton widget and found that while it worked under IE6, it was quite broken on Mozilla/Firebird.
If you look at the example, dynapi3x/examples/dynapi.gui.htmlradiobutton.html, when you click on the "Red" radio button you'll get an popup that says "Red Car" on IE but says "Blue Car" on Mozilla. It should say "Red Car" on both.
A quick look at the constructor for this class shows on line 15:
this._elmId = elmName||('Radio_'+this._group);
The "elmName" is not supplied to the constructor (at least in the example), and given that the element id should always be unique, I have changed the line to:
this._elmId = elmName||(this.id+'Radio');
- this is similar to what is in htmlcheckbox.js.
And on line 65, I changed the name of the radio button group from "this._elmId" to "this._group":
return '<input type="radio" ... +'" name="'+this._elmId+ ...
becomes:
return '<input type="radio" ... +'" name="'+this._group+' ...
This change seems to work quite well on both IE and Mozilla, so if there is a committer out there who is willing to commit this fix, that would be good.
Andrew.
------------------------------------------------------- This SF.Net email is sponsored by BEA Weblogic Workshop FREE Java Enterprise J2EE developer tools! Get your free copy of BEA WebLogic Workshop 8.1 today. http://ads.osdn.com/?ad_id=4721&alloc_id=10040&op=click _______________________________________________ Dynapi-Dev mailing list [EMAIL PROTECTED] http://www.mail-archive.com/[EMAIL PROTECTED]/
