From ad5cefeefc2dec285f9f9bb177e5bd190d0e6996 Mon Sep 17 00:00:00 2001 From: Aleksander Machniak <alec@alec.pl> Date: Fri, 22 Mar 2013 14:07:20 -0400 Subject: [PATCH] Better fix for IE9 issue with document.activeElement error (#1489008) --- program/js/editor.js | 31 +++++++++++++++++++++---------- 1 files changed, 21 insertions(+), 10 deletions(-) diff --git a/program/js/editor.js b/program/js/editor.js index e95ead1..3782de8 100644 --- a/program/js/editor.js +++ b/program/js/editor.js @@ -12,8 +12,6 @@ +-----------------------------------------------------------------------+ | Author: Eric Stadtherr <estadtherr@gmail.com> | +-----------------------------------------------------------------------+ - - $Id: editor.js 000 2006-05-18 19:12:28Z roundcube $ */ // Initialize HTML editor @@ -29,6 +27,7 @@ theme_advanced_toolbar_location: 'top', theme_advanced_toolbar_align: 'left', theme_advanced_buttons3: '', + theme_advanced_statusbar_location: 'none', extended_valid_elements: 'font[face|size|color|style],span[id|class|align|style]', relative_urls: false, remove_script_host: false, @@ -44,17 +43,26 @@ theme_advanced_buttons1: 'bold,italic,underline,strikethrough,justifyleft,justifycenter,justifyright,justifyfull,separator,outdent,indent,charmap,hr,link,unlink,code,forecolor', theme_advanced_buttons2: ',fontselect,fontsizeselect' }); - else // mail compose + else { // mail compose $.extend(conf, { - plugins: 'paste,emotions,media,nonbreaking,table,searchreplace,visualchars,directionality,tabfocus' + (config.spellcheck ? ',spellchecker' : ''), + plugins: 'paste,emotions,media,nonbreaking,table,searchreplace,visualchars,directionality,inlinepopups,tabfocus' + (config.spellcheck ? ',spellchecker' : ''), theme_advanced_buttons1: 'bold,italic,underline,|,justifyleft,justifycenter,justifyright,justifyfull,|,bullist,numlist,outdent,indent,ltr,rtl,blockquote,|,forecolor,backcolor,fontselect,fontsizeselect', - theme_advanced_buttons2: 'link,unlink,table,|,emotions,charmap,image,media,|,code,search' + (config.spellcheck ? ',spellchecker' : '') + ',undo,redo', + theme_advanced_buttons2: 'link,unlink,table,|,emotions,charmap,image,media,|,code,search,undo,redo', spellchecker_languages: (rcmail.env.spellcheck_langs ? rcmail.env.spellcheck_langs : 'Dansk=da,Deutsch=de,+English=en,Espanol=es,Francais=fr,Italiano=it,Nederlands=nl,Polski=pl,Portugues=pt,Suomi=fi,Svenska=sv'), - spellchecker_rpc_url: '?_task=utils&_action=spell_html', + spellchecker_rpc_url: '?_task=utils&_action=spell_html&_remote=1', spellchecker_enable_learn_rpc: config.spelldict, accessibility_focus: false, oninit: 'rcmail_editor_callback' }); + + // add handler for spellcheck button state update + conf.setup = function(ed) { + ed.onSetProgressState.add(function(ed, active) { + if (!active) + rcmail.spellcheck_state(); + }); + } + } // support external configuration settings e.g. from skin if (window.rcmail_editor_settings) @@ -72,12 +80,15 @@ if (rcmail.env.default_font) $(tinyMCE.get(rcmail.env.composebody).getBody()).css('font-family', rcmail.env.default_font); - if (elem && elem.type == 'select-one') { + if (elem && elem.type == 'select-one' && !rcmail.env.opened_extwin) { rcmail.change_identity(elem); // Focus previously focused element if (fe && fe.id != rcmail.env.composebody) { - window.focus(); // for WebKit (#1486674) - fe.focus(); + // use setTimeout() for IE9 (#1488541) + window.setTimeout(function() { + window.focus(); // for WebKit (#1486674) + fe.focus(); + }, 10); } } @@ -98,7 +109,7 @@ if (textarea && node) node.tabIndex = textarea.tabIndex; if (focus) - editor.getWin().focus(); + editor.getBody().focus(); } } } -- Gitblit v1.9.1