- Improve r4038: we can use keypress on FF and Opera, here we've got repetition

- Small code cleanup


git-svn-id: https://svn.roundcube.net/trunk@4039 208e9e7b-5314-0410-a742-e7e81cd9613c

2 files changed, 9 insertions, 6 deletions

diff --git a/roundcubemail/program/js/app.js b/roundcubemail/program/js/app.js
index 3a2cb08bf..93db9b93e 100644
--- a/roundcubemail/program/js/app.js
+++ b/roundcubemail/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/roundcubemail/program/js/editor.js b/roundcubemail/program/js/editor.js
index d3e587b75..1896e850f 100644
--- a/roundcubemail/program/js/editor.js
+++ b/roundcubemail/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();
 }