Fix regression where title attribute for long message subjects wasn't used ...

Aleksander Machniak

2014-04-14 7a5c3a3224bb59740aafceae89d236b19c2d8808

 program/js/app.js

@@ -1936,7 +1936,7 @@
    // add each submitted col
    for (n in this.env.listcols) {
      c = this.env.listcols[n];
      col = { className: String(c).toLowerCase() };
      col = {className: String(c).toLowerCase(), events:{}};

      if (this.env.coltypes[c] && this.env.coltypes[c].hidden) {
        col.className += ' hidden';
@@ -1970,11 +1970,8 @@
      else if (c == 'threads')
        html = expando;
      else if (c == 'subject') {
        if (bw.ie) {
          col.onmouseover = function() { rcube_webmail.long_subject_title_ex(this, message.depth+1); };
          if (bw.ie8)
            tree = '<span></span>' + tree; // #1487821
        }
        if (bw.ie)
          col.events.mouseover = function() { rcube_webmail.long_subject_title_ex(this); };
        html = tree + cols[c];
      }
      else if (c == 'priority') {
@@ -2180,7 +2177,7 @@
    var lock = this.set_busy(true, 'checkingmail'),
      params = this.check_recent_params();

    this.http_request('check-recent', params, lock);
    this.http_post('check-recent', params, lock);
  };

  // list messages of a specific mailbox using filter
@@ -4102,7 +4099,7 @@
    li.attr('id', name)
      .addClass(att.classname)
      .html(att.html)
      .on('mouseover', function() { rcube_webmail.long_subject_title_ex(this, 0); });
      .on('mouseover', function() { rcube_webmail.long_subject_title_ex(this); });

    // replace indicator's li
    if (upload_id && (indicator = document.getElementById(upload_id))) {
@@ -6829,13 +6826,13 @@

    if (action)
      query._action = action;
    else
    else if (this.env.action)
      query._action = this.env.action;

    var base = this.env.comm_path, k, param = {};

    // overwrite task name
    if (query._action.match(/([a-z0-9_-]+)\/([a-z0-9-_.]+)/)) {
    if (action && action.match(/([a-z0-9_-]+)\/([a-z0-9-_.]+)/)) {
      query._action = RegExp.$2;
      base = base.replace(/\_task=[a-z0-9_-]+/, '_task='+RegExp.$1);
    }
@@ -7500,7 +7497,7 @@
    this.env.lastrefresh = new Date();

    // plugins should bind to 'requestrefresh' event to add own params
    this.http_request('refresh', params, lock);
    this.http_post('refresh', params, lock);
  };

  // returns check-recent request parameters
@@ -7862,7 +7859,7 @@
  }
};

rcube_webmail.long_subject_title_ex = function(elem, indent)
rcube_webmail.long_subject_title_ex = function(elem)
{
  if (!elem.title) {
    var $elem = $(elem),
@@ -7874,7 +7871,7 @@
      w = tmp.width();

    tmp.remove();
    if (w + indent * 15 > $elem.width())
    if (w + $('span.branch', $elem).width() * 15 > $elem.width())
      elem.title = txt;
  }
};