- don't use onchange to toggle editor because of IE bug

alecpl

2008-04-16 3dd0f253339e7f8e2aef6567987709c07b5d4e7f

- don't use onchange to toggle editor because of IE bug

 program/js/editor.js

@@ -35,26 +35,6 @@
               });
  }

// Set the state of the HTML/Plain toggles based on the _is_html field value
function rcmail_set_editor_toggle_states()
  {
  // set the editor toggle based on the state of the editor

  var htmlFlag = document.getElementsByName('_is_html')[0];
  var toggles = document.getElementsByName('_editorSelect');
  for(var t=0; t<toggles.length; t++)
    {
    if (toggles[t].value == 'html')
      {
      toggles[t].checked = (htmlFlag.value == "1");
      }
    else
      {
      toggles[t].checked = (htmlFlag.value == "0");
      }
    }
  }

// Toggle between the HTML and Plain Text editors

function rcmail_toggle_editor(toggler)
@@ -62,7 +42,6 @@
  var selectedEditor = toggler.value;

  // determine the currently displayed editor

  var htmlFlag = document.getElementsByName('_is_html')[0];
  var isHtml = htmlFlag.value;


 program/steps/mail/compose.inc

@@ -825,7 +825,7 @@
  $selector = '';
  
  $attrib['name'] = '_editorSelect';
  $attrib['onchange'] = 'return rcmail_toggle_editor(this)';
  $attrib['onclick'] = 'return rcmail_toggle_editor(this)';
  foreach ($choices as $value => $text)
  {
    $attrib['id'] = '_' . $value;