Combo box code is Working in IE perfectly but it is not working in FireFox

[buss123]

Hi all,
combo box script code was working in IE perfectly with all modes but OnChange event was not working in FireFox(editabl e mode, if we select valuese that combo box values r not dislaying if enter the data the it is inserting properly) javascript code follows

pls help me to solve this problem

  function fnKeyDownHandler(getdropdown, e)
  {
    fnSanityCheck(getdropdown);
 
    var vEventKeyCode = FindKeyCode(e);
    if(vEventKeyCode == 37)
    {
    fnLeftToRight(getdropdown);
    }
    if(vEventKeyCode == 39)
    {
    fnRightToLeft(getdropdown);
    }
    if(vEventKeyCode == 46)
    {
    fnDelete(getdropdown);
    }
 
    if(vEventKeyCode == 8 || vEventKeyCode==127)
    {
    if(e.which) 
    {
 
    }
    else 
    {
 
      e.keyCode = '';
      if(window.event.keyCode)
      {
      window.event.keyCode = '';
      }
    }
    return true;
    }
 
  }
 
  function fnLeftToRight(getdropdown)
  {
    getdropdown.style.direction = "ltr";
  }
 
  function fnRightToLeft(getdropdown)
  {
    getdropdown.style.direction = "rtl";
  }
 
  function fnDelete(getdropdown)
  {
    if(getdropdown.options.length != 0)
 
    {
    if (getdropdown.options.selectedIndex == vEditableOptionIndex_A)
 
    {
      getdropdown.options[getdropdown.options.selectedIndex].text = '';
 
    }
    }
  }
  function FindKeyCode(e)
  {
    if(e.which)
    {
    keycode=e.which; 
    }
    else
    {
    keycode=e.keyCode; 
    }
    return keycode;
  }
 
  function FindKeyChar(e)
  {
    keycode = FindKeyCode(e);
    if((keycode==8)||(keycode==127))
    {
    character="backspace"
    }
    else if((keycode==46))
    {
    character="delete"
    }
    else
    {
    character=String.fromCharCode(keycode);
    }
 
    return character;
  }
 
  function fnSanityCheck(getdropdown)
  {
    if(vEditableOptionIndex_A>(getdropdown.options.length-1))
    {
    alert("PROGRAMMING ERROR: The value of variable vEditableOptionIndex_... cannot be greater than (length of dropdown - 1)");
    return false;
    }
  }
  var vEditableOptionIndex_A = 0;
 
  var vEditableOptionText_A = "--?--";
 
 
 
  var vPreviousSelectIndex_A = 0;
 
  var vSelectIndex_A = 0;
 
  var vSelectChange_A = 'MANUAL_CLICK';
 
 
  function fnChangeHandler_A(getdropdown)
  {
    fnSanityCheck(getdropdown);
 
    vPreviousSelectIndex_A = vSelectIndex_A;
 
    vSelectIndex_A = getdropdown.options.selectedIndex;
 
 
    if ((vPreviousSelectIndex_A == (vEditableOptionIndex_A)) && (vSelectIndex_A != (vEditableOptionIndex_A))&&(vSelectChange_A != 'MANUAL_CLICK'))
 
    {
      getdropdown[(vEditableOptionIndex_A)].selected=true;
      vPreviousSelectIndex_A = vSelectIndex_A;
      vSelectIndex_A = getdropdown.options.selectedIndex;
      vSelectChange_A = 'MANUAL_CLICK';
 
    }
  }
 
  function fnKeyPressHandler_A(getdropdown, e)
  {
    fnSanityCheck(getdropdown);
 
    keycode = FindKeyCode(e);
    keychar = FindKeyChar(e);
 
 
 
    if ((keycode>47 && keycode<59)||(keycode>62 && keycode<127) ||(keycode==32) ||(keycode==46))
    {
      var vAllowableCharacter = "yes";
    }
    else
    {
      var vAllowableCharacter = "no";
    }
    if(getdropdown.options.length != 0)
      if (getdropdown.options.selectedIndex == (vEditableOptionIndex_A))
            {
        var vEditString = getdropdown[vEditableOptionIndex_A].text;
        if((vAllowableCharacter == "yes")||(keychar=="backspace"))
        {
          if (vEditString == vEditableOptionText_A)
            vEditString = "";
        }
        if (keychar=="backspace")
 
        {
          vEditString = vEditString.substring(0,vEditString.length-1);
 
          vSelectChange_A = 'MANUAL_CLICK';
                 }
        if (vAllowableCharacter == "yes")
 
        {
          vEditString+=String.fromCharCode(keycode);          
          var i=0;
          var vEnteredChar = String.fromCharCode(keycode);
          var vUpperCaseEnteredChar = vEnteredChar;
          var vLowerCaseEnteredChar = vEnteredChar;
          if(((keycode)>=97)&&((keycode)<=122))
 
            vUpperCaseEnteredChar = String.fromCharCode(keycode - 32);
 
          if(((keycode)>=65)&&((keycode)<=90))
 
            vLowerCaseEnteredChar = String.fromCharCode(keycode + 32);
 
          if(e.which) 
          {
 
            for (i=0;i<=(getdropdown.options.length-1);i++)
            {
              if(i!=vEditableOptionIndex_A)
              {
                var vReadOnlyString = getdropdown[i].text;
                var vFirstChar = vReadOnlyString.substring(0,1);
                if((vFirstChar == vUpperCaseEnteredChar)||(vFirstChar == vLowerCaseEnteredChar))
                {
                  vSelectChange_A = 'AUTO_SYSTEM';
 
                  break;
                }
                else
                {
                  vSelectChange_A = 'MANUAL_CLICK';
 
                }
              }
            }
          }
        }
 
 
        getdropdown.options[vEditableOptionIndex_A].text = vEditString;
 
        getdropdown.options[vEditableOptionIndex_A].value = vEditString; //Use this line only if want to change the internal value too; else this line is not required.
 
        return false;
      }
    return true;
  }
 
  function fnKeyUpHandler_A(getdropdown, e)
  {
    fnSanityCheck(getdropdown);
 
    if(e.which) 
    {
      if(vSelectChange_A == 'AUTO_SYSTEM')
      {
 
        getdropdown[(vEditableOptionIndex_A)].selected=true;
      }
 
      var vEventKeyCode = FindKeyCode(e);
 
      if((vEventKeyCode == 37)||(vEventKeyCode == 39))
      {
        getdropdown[vEditableOptionIndex_A].selected=true;
      }
    }
  }

Please post code using code tags - moderator

pls help me i am not able to do this

[buss123]

What's "select-one"? The type of a combo box is going to be "select".

Do you get any errors? If so, on what line?

select-one is the combobox name . we didnt get any errors . combobox working fine but the values are not displaying in the FireFox .


Thanking u

[acoder]

The name of the combo box that you're passing is "MedDirecti on0"

[voxecho]

Ok, i know this comes a little late on the topic, but i had to comment on this.

build a select box (size=1 line)
give it some dummy options
now, add the following

onchange="alert (this.type)"

any guess what it will return? i will give you a hint, it's not "select"

it will return "select-one" and if it's a multiple select it will return... yes, you guessed it "select-multiple"

so checking on the o.type=="select-one" is perfectly valid;

just thought i would weigh in on that.

Instead of

if(document.formmain(comboName).type=='select-one'){

try:

if(document.formmain[comboName].name=='select-one'){

Surely, you mean the name, not the type.

Perhaps you could post your form HTML as well as how you are calling this function.

[voxecho]

bus, did you ever solve this?

i wonder if the problem isn't the fact that you are addressing the element with the wrong indexing operator... actually i am surprised it's working in any broswer

rather then

document.formname(formelement).type==

try

document.formname[formElement].type==

note the [ rather then (

select-one is the combobox name . we didnt get any errors . combobox working fine but the values are not displaying in the FireFox .


Thanking u

[gits]

Ok, i know this comes a little late on the topic, but i had to comment on this.

build a select box (size=1 line)
give it some dummy options
now, add the following

onchange="alert (this.type)"

any guess what it will return? i will give you a hint, it's not "select"

it will return "select-one" and if it's a multiple select it will return... yes, you guessed it "select-multiple"

so checking on the o.type=="select-one" is perfectly valid;

just thought i would weigh in on that.

;) interesting hint i tested it and of course you're right ... i didn't know about that (and never needed it yet but good to know now) ... thank you for this ...

kind regards ...

[acoder]

Ok, i know this comes a little late on the topic, but i had to comment on this.

build a select box (size=1 line)
give it some dummy options
now, add the following

onchange="alert (this.type)"

any guess what it will return? i will give you a hint, it's not "select"

it will return "select-one" and if it's a multiple select it will return... yes, you guessed it "select-multiple"

so checking on the o.type=="select-one" is perfectly valid;

just thought i would weigh in on that.

Interesting - I didn't know that. Thanks for pointing that out.

Incidentally, I found this - silly me for not checking.

[buss123]

Hi all,
the following line of code is working fine in IE but the same line of code is not working in Firefox . can any one help me is there any other way to write the same line of code for firefox.

if(document.for mmain(comboName ).type=='select-one')

my main doubt is 'select-one' will work in firefox or not? if not pls tell me is there any alternative for that.

[acoder]

This reminds me of your earlier thread. Is it the same problem?

[buss123]

This reminds me of your earlier thread. Is it the same problem?

Yes after a long time again i shifteed to that problem again , what is the compatiable word to "select-one"

[Logician]

Yes after a long time again i shifteed to that problem again , what is the compatiable word to "select-one"

You still don't seem to understand, the problem has nothing to do with the type identifier.

The syntax

document.formmain(comboName)

is incorrect as it constitutes a function call not a reference. The Firefox error console must be indicating this.

if(document.formmain[comboName].type=='select-one')