I have changed that part to 'change' but it is still not working there
is some issue with the showOrHide part which I am unable to figure out
....because in Firefox the alert ("hi 2"); is not coming up and that is
the code for showOrHide
<script language="JavaScript">
var cntlName;
var eleTarget = document.getElementById('hiding');
function showOrHide(){
var eleSource = event.srcElement;
if (event.srcElement) {
//for IE
alert ("hi 1");
var eleSource = event.srcElement;
}
else {
//for Firefox
alert ("hi 2");
var eleSource = event.target;
}
if (eleSource.value == 'MGRPAY') {
//for IE
eleTarget.style.display = 'block';
}
else {
eleTarget.style.display = 'none';
}
}
for(var i=0; i<preProcessControlArray.length; i++){
cntlName = eval(preProcessControlArray[i]);
if (cntlName.m_oSubmit.name.toLowerCase() == 'p_par_rad_role'){
if (cntlName.m_oForm.length) {
for( var j=0; j<cntlName.m_oForm.length; j++ ){
if (j == 0) {
cntlName.m_oForm[j].focus();
cntlName.m_oForm[j].click();
if (cntlName.m_oForm[j].value == 'MGRPAY') {
eleTarget.style.display = 'block';
}
else {
eleTarget.style.display = 'none';
}
}
//For IE
if(cntlName.m_oForm[j].attachEvent)
cntlName.m_oForm[j].attachEvent('onpropertychange',
showOrHide);
//for firefox
cntlName.m_oForm[j].addEventListener('change',showOrHide,false);
}
}
else {
cntlName.m_oForm.focus();
cntlName.m_oForm.click();
if (cntlName.m_oForm.value == 'MGRPAY') {
eleTarget.style.display = 'block';
}
else {
eleTarget.style.display = 'none';
}
}
}
}
</script>
Richard Cornford wrote:[color=blue]
> Coder wrote:[color=green]
> > Hi I have the following code in java script, it is not giving
> > proper output in FIREFOX but running fine in IE... can anybody
> > help me out to make this run in FIREFOX .
> >
> > <script language="JavaScript">
> > var cntlName;
> > var eleTarget = document.getElementById('hiding');
> >
> > function showOrHide(){
> > alert ("hi");
> > var eleSource = event.srcElement;[/color]
> ^^^^^
> Firefox/Mozilla/Gecko browsers do not use an global - event - property,
> instead passing event objects as the argument to the event handling
> function. Because the - event - Identifier does not refer to a defined
> property of the global object it evaluates as the undefined value, and
> the subsequent attempt to read the - srcElement - property of an
> undefined value will have firefox error-out.Firefox/Mozilla/Gecko
> browsers do not use an global - event - property, instead passing event
> objects as the argument to the event handling function. Because the -
> event - Identifier does not refer to a defined property of the global
> object it evaluates as the undefined value, and the subsequent attempt
> to read the - srcElement - property of an undefined value will have
> firefox error-out.
>[color=green]
> > if (event.srcElement) {
> > //for IE
> > var eleSource = event.srcElement;
> > }
> > else {
> > //for Firefox
> > var eleSource = event.target;
> > }
> >
> > if (eleSource.value == 'MGRPAY') {
> > //for IE
> > eleTarget.style.display = 'block';[/color]
>
> If - eleSource - is an input element, as the use of its value property
> may imply, then its default CSS display style is 'inline' not 'block'.
>[color=green]
> > }
> > else {
> > eleTarget.style.display = 'none';
> > }
> > }[/color]
>
> function showOrHide(ev){
> var eleSource;
> ev = ev || window.event;
> if(eleSource = (ev.srcElement||ev.target)){
> if(eleSource.value == 'MGRPAY'){
> eleTarget.style.display = '';
> }else{
> eleTarget.style.display = 'none';
> }
> }
> }
>[color=green]
> > for(var i=0; i<preProcessControlArray.length; i++){
> > cntlName = eval(preProcessControlArray[i]);[/color]
>
> The contents of - preProcessControlArray - are not shown here so what
> this code does, or may be intended to do, is unknown. Though generally
> any use of - eval - is indicative of failing to do something else what
> would be objectively better.
>
> However, if this array contains control names in the form of string
> then - eval -ing them will be equivalent to resolving Identifiers as
> properties on the scope chain, or more likely the global object. IE
> makes (at least some) named elements into properties of the global
> object that refer to the elements. Firefox/Mozilla/Gecko browsers do not
> tend to do this.
>[color=green]
> > if (cntlName.m_oSubmit.name.toLowerCase() == 'p_par_rad_role'){
> > if (cntlName.m_oForm.length) {
> > for( var j=0; j<cntlName.m_oForm.length; j++ ){
> > if (j == 0) {
> > cntlName.m_oForm[j].focus();
> > cntlName.m_oForm[j].click();
> > if (cntlName.m_oForm[j].value == 'MGRPAY') {
> > eleTarget.style.display = 'block';
> > }
> > else {
> > eleTarget.style.display = 'none';
> > }
> > }
> > //For IE
> > if(cntlName.m_oForm[j].attachEvent)
> > cntlName.m_oForm[j].attachEvent('onpropertychange',
> > showOrHide);
> > //for firefox
> > else
> > cntlName.m_oForm[j].addEventListener('onchange',[/color]
> <snip> ^^^^^^^^
> The addEventListener method takes the event name as its argument not the
> name of the intrinsic event property, so here it would be 'change'
> rather than 'onchange'.
>
> The onchange hander is not triggered under equivalent conditions to the
> onpropertychange even in IE, and IE does support onchange.
>
> Richard.[/color]
