Fix mouse selection on autocomplete lists

Aleksander Machniak

2014-06-08 1f2699675d53019a9e42bfa7b60de1f23812b39d

 program/js/app.js

@@ -46,12 +46,12 @@
  this.messages = {};
  this.group2expand = {};
  this.http_request_jobs = {};
  this.menu_stack = new Array();
  this.menu_stack = [];

  // webmail client settings
  this.dblclick_time = 500;
  this.message_time = 5000;
  this.identifier_expr = new RegExp('[^0-9a-z\-_]', 'gi');
  this.identifier_expr = /[^0-9a-z_-]/gi;

  // environment defaults
  this.env = {
@@ -314,9 +314,9 @@
              });

            // avoid textarea loosing focus when hitting the save-response button/link
            for (var i=0; this.buttons['save-response'] && i < this.buttons['save-response'].length; i++) {
              $('#'+this.buttons['save-response'][i].id).mousedown(function(e){ return rcube_event.cancel(e); })
            }
            $.each(this.buttons['save-response'] || [], function (i, v) {
              $('#' + v.id).mousedown(function(e){ return rcube_event.cancel(e); })
            });
          }

          // init message compose form
@@ -403,9 +403,6 @@
            .addEventListener('dragend', function(e) { ref.drag_end(e); })
            .init();

          if (this.env.cid)
            this.contact_list.highlight_row(this.env.cid);

          this.gui_objects.contactslist.parentNode.onmousedown = function(e){ return ref.click_on_list(e); };

          $(this.gui_objects.qsearchbox).focusin(function() { ref.contact_list.blur(); });
@@ -471,9 +468,6 @@
            })
            .init()
            .focus();

          if (this.env.iid)
            this.identity_list.highlight_row(this.env.iid);
        }
        else if (this.gui_objects.sectionslist) {
          this.sections_list = new rcube_list_widget(this.gui_objects.sectionslist, {multiselect:false, draggable:false, keyboard:true});
@@ -1344,7 +1338,7 @@
  this.command_enabled = function(cmd)
  {
    return this.commands[cmd];
  }
  };

  // lock/unlock interface
  this.set_busy = function(a, message, id)
@@ -1386,10 +1380,11 @@
  // switch to another application task
  this.switch_task = function(task)
  {
    if (this.task===task && task!='mail')
    if (this.task === task && task != 'mail')
      return;

    var url = this.get_task_url(task);

    if (task == 'mail')
      url += '&_mbox=INBOX';
    else if (task == 'logout')
@@ -1444,10 +1439,10 @@

  this.save_pref = function(prop)
  {
    var request = {'_name': prop.name, '_value': prop.value};
    var request = {_name: prop.name, _value: prop.value};

    if (prop.session)
      request['_session'] = prop.session;
      request._session = prop.session;
    if (prop.env)
      this.env[prop.env] = prop.value;

@@ -1587,7 +1582,7 @@

      // select the folder if one of its childs is currently selected
      // don't select if it's virtual (#1488346)
      if (this.env.mailbox && this.env.mailbox.startsWith(name + this.env.delimiter) && !node.virtual)
      if (!node.virtual && this.env.mailbox && this.env.mailbox.startsWith(name + this.env.delimiter))
        this.command('list', name);
    }
    else {
@@ -1629,7 +1624,7 @@
    }

    // reset popup menus; delayed to have updated menu_stack data
    window.setTimeout(function(e){
    setTimeout(function(e){
      var obj, skip, config, id, i, parents = $(target).parents();
      for (i = ref.menu_stack.length - 1; i >= 0; i--) {
        id = ref.menu_stack[i];
@@ -1669,6 +1664,9 @@
    var target = e.target || {},
      keyCode = rcube_event.get_keycode(e);

    // save global reference for keyboard detection on click events in IE
    rcube_event._last_keyboard_event = e;

    if (e.keyCode != 27 && (!this.menu_keyboard_active || target.nodeName == 'TEXTAREA' || target.nodeName == 'SELECT')) {
      return true;
    }
@@ -1678,7 +1676,7 @@
      case 40:
      case 63232: // "up", in safari keypress
      case 63233: // "down", in safari keypress
        focus_menu_item(mod = keyCode == 38 || keyCode == 63232 ? -1 : 1);
        focus_menu_item(keyCode == 38 || keyCode == 63232 ? -1 : 1);
        break;

      case 9:   // tab
@@ -2428,6 +2426,9 @@
      url._framed = 1;
    }

    if (this.env.uid)
      url._uid = this.env.uid;

    // load message list to target frame/window
    if (mbox) {
      this.set_busy(true, 'loading');
@@ -2474,7 +2475,7 @@
        selection.push(selected[i]);

    this.message_list.selection = selection;
  }
  };

  // expand all threads with unread children
  this.expand_unread = function()
@@ -2487,8 +2488,10 @@
        this.message_list.expand_all(r);
        this.set_unread_children(r.uid);
      }

      new_row = new_row.nextSibling;
    }

    return false;
  };

@@ -2705,8 +2708,8 @@
    }

    // update unread_children for roots
    for (var i=0; i<roots.length; i++)
      this.set_unread_children(roots[i].uid);
    for (r=0; r<roots.length; r++)
      this.set_unread_children(roots[r].uid);

    return count;
  };
@@ -3706,8 +3709,9 @@
  {
    var active = this.editor.spellcheck_state();

    if (this.buttons.spellcheck)
      $('#'+this.buttons.spellcheck[0].id)[active ? 'addClass' : 'removeClass']('selected');
    $.each(this.buttons.spellcheck || [], function(i, v) {
      $('#' + v.id)[active ? 'addClass' : 'removeClass']('selected');
    });

    return active;
  };
@@ -3905,7 +3909,7 @@
      }
      this.local_storage_remove_item('compose.index');
    }
  }
  };


  this.change_identity = function(obj, show_sig)
@@ -4119,7 +4123,7 @@

  this.upload_progress_update = function(param)
  {
    var elem = $('#'+param.name + '> span');
    var elem = $('#'+param.name + ' > span');

    if (!elem.length || !param.text)
      return;
@@ -4513,7 +4517,7 @@

      // register (delegate) event handlers
      ul.on('mouseover', 'li', function(e){ ref.ksearch_select(e.target); })
        .on('onmouseup', 'li', function(e){ ref.ksearch_click(e.target); })
        .on('mouseup', 'li', function(e){ ref.ksearch_click(e.target); })
    }

    ul = this.ksearch_pane.__ul;
@@ -4968,7 +4972,7 @@
  {
    var selection = this.contact_list ? this.contact_list.get_selection() : [];

    // exit if no mailbox specified or if selection is empty
    // exit if no contact specified or if selection is empty
    if (!selection.length && !this.env.cid)
      return;

@@ -7255,11 +7259,24 @@
          this.enable_command('expand-all', 'expand-unread', 'collapse-all', this.env.threading && this.env.messagecount && !is_multifolder);

          if ((response.action == 'list' || response.action == 'search') && this.message_list) {
            var list = this.message_list, uid = this.env.list_uid;

            // highlight message row when we're back from message page
            if (uid) {
              if (!list.rows[uid])
                uid += '-' + this.env.mailbox;
              if (list.rows[uid]) {
                list.select(uid);
              }
              delete this.env.list_uid;
            }

            this.enable_command('set-listmode', this.env.threads && !is_multifolder);
            if (this.message_list.rowcount > 0)
              this.message_list.focus();
            this.msglist_select(this.message_list);
            this.triggerEvent('listupdate', { folder:this.env.mailbox, rowcount:this.message_list.rowcount });
            if (list.rowcount > 0)
              list.focus();
            this.msglist_select(list);
            this.triggerEvent('listupdate', { folder:this.env.mailbox, rowcount:list.rowcount });

          }
        }
        else if (this.task == 'addressbook') {