From 65e735dec541564c01defb58eac326e7369feb7e Mon Sep 17 00:00:00 2001 From: Aleksander Machniak Date: Thu, 12 Mar 2015 20:48:19 +0100 Subject: Fix javascripts errors in IE8 - lack of Event.which, focusing a hidden element (#1490318) --- program/js/app.js | 11 +++++++---- program/js/list.js | 4 ++-- 2 files changed, 9 insertions(+), 6 deletions(-) (limited to 'program') diff --git a/program/js/app.js b/program/js/app.js index 5f3ca5f65..d6c5cd8aa 100644 --- a/program/js/app.js +++ b/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; diff --git a/program/js/list.js b/program/js/list.js index 9a7f2840e..9c41a12da 100644 --- a/program/js/list.js +++ b/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) { -- cgit v1.2.3