Fix javascripts errors in IE8 - lack of Event.which, focusing a hidden element (#1490318)

CHANGELOG

@@ -36,6 +36,7 @@ CHANGELOG Roundcube Webmail
 - Fix backtick character handling in sql queries (#1490312)
 - Fix redirct URL for attachments loaded in an iframe when behind proxy (#1490191)
 - Fix menu container references to point to the actual <ul> element (#1490313)
+- Fix javascripts errors in IE8 - lack of Event.which, focusing a hidden element (#1490318)
 
 RELEASE 1.1.0
 -------------

program/js/app.js

@@ -3349,7 +3349,7 @@ function rcube_webmail()
     if (!this.gui_objects.messageform)
       return false;
 
-    var i, pos, input_from = $("[name='_from']"),
+    var i, elem, pos, input_from = $("[name='_from']"),
       input_to = $("[name='_to']"),
       input_subject = $("input[name='_subject']"),
       input_message = $("[name='_message']").get(0),
@@ -3403,11 +3403,14 @@ function rcube_webmail()
       this.compose_restore_dialog(0, html_mode)
 
     if (input_to.val() == '')
-      input_to.focus();
+      elem = input_to;
     else if (input_subject.val() == '')
-      input_subject.focus();
+      elem = input_subject;
     else if (input_message)
-      input_message.focus();
+      elem = input_message;
+
+    // focus first empty element (need to be visible on IE8)
+    $(elem).filter(':visible').focus();
 
     this.env.compose_focus_elem = document.activeElement;
 

program/js/list.js

@@ -151,8 +151,8 @@ init_row: function(row)
     this.rows[uid] = {uid:uid, id:row.id, obj:row};
 
     // set eventhandlers to table row (only left-button-clicks in mouseup)
-    row.onmousedown = function(e){ return self.drag_row(e, this.uid); };
-    row.onmouseup = function(e){ if (e.which == 1) return self.click_row(e, this.uid); };
+    $(row).mousedown(function(e) { return self.drag_row(e, this.uid); })
+      .mouseup(function(e) { if (e.which == 1) return self.click_row(e, this.uid); });
 
     if (bw.touch && row.addEventListener) {
       row.addEventListener('touchstart', function(e) {