Fix keyboard events on list widgets in Internet Explorer (#1489025) - It wa...

Aleksander Machniak

2013-03-28 0c974b263d8fbd1f2ab3557b3b860442c0c277cd

Fix keyboard events on list widgets in Internet Explorer (#1489025) - It was a regression in fix for #1489008

 CHANGELOG

@@ -1,6 +1,10 @@
CHANGELOG Roundcube Webmail
===========================

- Fix keyboard events on list widgets in Internet Explorer (#1489025)

RELEASE 0.9-rc2
---------------
- Fix security issue in save-pref command
- Remove sig_above configuration option, use reply_mode only (#1489001)
- Refresh current folder in opener window after draft save or message sent (#1488997)

 program/js/app.js

@@ -178,11 +178,6 @@
      parent.rcmail.env.frame_lock = null;
    }

    // Makes that reference to document.activeElement do not throw
    // "unspecified error" in IE9 (#1489008)
    if (this.env.framed && bw.ie)
      document.documentElement.focus();

    // enable general commands
    this.enable_command('close', 'logout', 'mail', 'addressbook', 'settings', 'save-pref',
      'compose', 'undo', 'about', 'switch-task', 'menu-open', 'menu-save', true);

 program/js/common.js

@@ -717,7 +717,7 @@
// jQuery plugin to emulate HTML5 placeholder attributes on input elements
jQuery.fn.placeholder = function(text) {
  return this.each(function() {
    var elem = $(this);
    var active = false, elem = $(this);
    this.title = text;

    // Try HTML5 placeholder attribute first
@@ -742,8 +742,9 @@
        elem[(active ? 'addClass' : 'removeClass')]('placeholder').attr('spellcheck', active);
      });

      // Do not blur currently focused element
      if (this != document.activeElement)
      // Do not blur currently focused element (catch exception: #1489008)
      try { active = this == document.activeElement; } catch(e) {}
      if (!active)
        elem.blur();
    }
  });

 program/js/list.js

@@ -692,6 +692,7 @@
    this.shift_start = null;

  this.last_selected = id;
  this.list.focus();
},