diff options
author | alecpl <alec@alec.pl> | 2010-10-05 07:49:00 +0000 |
---|---|---|
committer | alecpl <alec@alec.pl> | 2010-10-05 07:49:00 +0000 |
commit | c288f9969e113cb7dbd38bf05167cc8e94e007ff (patch) | |
tree | 5f7b6f81a41fee3097d626712091a3fee7591452 | |
parent | 8cfbc47e895becc0af66d736ce5cc2bd7946feed (diff) |
- Improve r4038: we can use keypress on FF and Opera, here we've got repetition
- Small code cleanup
-rw-r--r-- | program/js/app.js | 2 | ||||
-rw-r--r-- | program/js/editor.js | 13 |
2 files changed, 9 insertions, 6 deletions
diff --git a/program/js/app.js b/program/js/app.js index 3a2cb08bf..93db9b93e 100644 --- a/program/js/app.js +++ b/program/js/app.js @@ -2768,7 +2768,7 @@ function rcube_webmail() this.init_address_input_events = function(obj) { - obj.keydown(function(e){ return ref.ksearch_keydown(e, this); }) + obj[bw.ie || bw.safari || bw.chrome ? 'keydown' : 'keypress'](function(e){ return ref.ksearch_keydown(e, this); }) .attr('autocomplete', 'off'); }; diff --git a/program/js/editor.js b/program/js/editor.js index d3e587b75..1896e850f 100644 --- a/program/js/editor.js +++ b/program/js/editor.js @@ -64,15 +64,18 @@ function rcmail_editor_init(skin_path, editor_lang, spellcheck, mode) // react to real individual tinyMCE editor init function rcmail_editor_callback() { - var elem = rcube_find_object('_from'); - if (elem && elem.type=='select-one') { + var elem = rcube_find_object('_from'), + fe = rcmail.env.compose_focus_elem; + + if (elem && elem.type == 'select-one') { rcmail.change_identity(elem); // Focus previously focused element - if (rcmail.env.compose_focus_elem && rcmail.env.compose_focus_elem.id != rcmail.env.composebody) - rcmail.env.compose_focus_elem.focus(); + if (fe && fe.id != rcmail.env.composebody) + fe.focus(); } + // set tabIndex and set focus to element that was focused before - rcmail_editor_tabindex(rcmail.env.compose_focus_elem && rcmail.env.compose_focus_elem.id == rcmail.env.composebody); + rcmail_editor_tabindex(fe && fe.id == rcmail.env.composebody); // Trigger resize (needed for proper editor resizing in some browsers using default skin) $(window).resize(); } |