- code formatting, cleanup and small improvements

alecpl

2010-04-16 8fa9229d578a9c2f265b4e46278c3f5f4329baae

 program/js/list.js

@@ -57,7 +57,7 @@
  if (p && typeof(p)=='object')
    for (var n in p)
      this[n] = p[n];
  }
};


rcube_list_widget.prototype = {
@@ -68,17 +68,14 @@
 */
init: function()
{
  if (this.list && this.list.tBodies[0])
  {
    this.rows = new Array();
  if (this.list && this.list.tBodies[0]) {
    this.rows = [];
    this.rowcount = 0;

    var row;
    for(var r=0; r<this.list.tBodies[0].childNodes.length; r++)
    {
    for(var r=0; r<this.list.tBodies[0].childNodes.length; r++) {
      row = this.list.tBodies[0].childNodes[r];
      while (row && row.nodeType != 1)
      {
      while (row && row.nodeType != 1) {
        row = row.nextSibling;
        r++;
      }
@@ -104,8 +101,7 @@
init_row: function(row)
{
  // make references in internal array and set event handlers
  if (row && String(row.id).match(/rcmrow([a-z0-9\-_=\+\/]+)/i))
  {
  if (row && String(row.id).match(/rcmrow([a-z0-9\-_=\+\/]+)/i)) {
    var p = this;
    var uid = RegExp.$1;
    row.uid = uid;
@@ -131,10 +127,11 @@
  var tbody = document.createElement('tbody');
  this.list.insertBefore(tbody, this.list.tBodies[0]);
  this.list.removeChild(this.list.tBodies[1]);
  this.rows = new Array();
  this.rows = [];
  this.rowcount = 0;
  
  if (sel) this.clear_selection();
  if (sel)
    this.clear_selection();
},


@@ -181,8 +178,7 @@
focus: function(e)
{
  this.focused = true;
  for (var n=0; n<this.selection.length; n++)
  {
  for (var n=0; n<this.selection.length; n++) {
    id = this.selection[n];
    if (this.rows[id] && this.rows[id].obj) {
      $(this.rows[id].obj).addClass('selected').removeClass('unfocused');
@@ -201,8 +197,7 @@
{
  var id;
  this.focused = false;
  for (var n=0; n<this.selection.length; n++)
  {
  for (var n=0; n<this.selection.length; n++) {
    id = this.selection[n];
    if (this.rows[id] && this.rows[id].obj) {
      $(this.rows[id].obj).removeClass('selected').addClass('unfocused');
@@ -229,14 +224,12 @@
  this.in_selection_before = this.in_selection(id) ? id : false;

  // selects currently unselected row
  if (!this.in_selection_before)
  {
  if (!this.in_selection_before) {
    var mod_key = rcube_event.get_modifier(e);
    this.select_row(id, mod_key, false);
  }

  if (this.draggable && this.selection.length)
  {
  if (this.draggable && this.selection.length) {
    this.drag_start = true;
    this.drag_mouse_start = rcube_event.get_mouse_pos(e);
    rcube_event.add_listener({event:'mousemove', object:this, method:'drag_mouse_move'});
@@ -272,8 +265,7 @@
    return true;

  // don't do anything (another action processed before)
  if (this.dont_select)
    {
  if (this.dont_select) {
    this.dont_select = false;
    return false;
    }
@@ -535,12 +527,12 @@

get_first_row: function()
{
  if (this.rowcount)
    {
  if (this.rowcount) {
    var rows = this.list.tBodies[0].rows;

    for(var i=0; i<rows.length-1; i++)
      if(rows[i].id && String(rows[i].id).match(/rcmrow([a-z0-9\-_=\+\/]+)/i) && this.rows[RegExp.$1] != null)

   return RegExp.$1;
    }

@@ -549,12 +541,12 @@

get_last_row: function()
{
  if (this.rowcount)
    {
  if (this.rowcount) {
    var rows = this.list.tBodies[0].rows;

    for(var i=rows.length-1; i>=0; i--)
      if(rows[i].id && String(rows[i].id).match(/rcmrow([a-z0-9\-_=\+\/]+)/i) && this.rows[RegExp.$1] != null)

   return RegExp.$1;
    }

@@ -574,16 +566,13 @@
  if (!this.shift_start)
    this.shift_start = id

  if (!mod_key)
  {
  if (!mod_key) {
    this.shift_start = id;
    this.highlight_row(id, false);
    this.multi_selecting = false;
  }
  else
  {
    switch (mod_key)
    {
  else {
    switch (mod_key) {
      case SHIFT_KEY:
        this.shift_select(id, false);
        break;
@@ -612,8 +601,7 @@
    $(this.rows[this.last_selected].obj).removeClass('focused');

  // unselect if toggleselect is active and the same row was clicked again
  if (this.toggleselect && this.last_selected == id)
  {
  if (this.toggleselect && this.last_selected == id) {
    this.clear_selection();
    id = null;
  }
@@ -722,16 +710,13 @@
  var j = ((from_rowIndex > to_rowIndex)? from_rowIndex : to_rowIndex);

  // iterate through the entire message list
  for (var n in this.rows)
  {
    if ((this.rows[n].obj.rowIndex >= i) && (this.rows[n].obj.rowIndex <= j))
    {
  for (var n in this.rows) {
    if ((this.rows[n].obj.rowIndex >= i) && (this.rows[n].obj.rowIndex <= j)) {
      if (!this.in_selection(n)) {
        this.highlight_row(n, true);
      }
    }
    else
    {
    else {
      if  (this.in_selection(n) && !control) {
        this.highlight_row(n, true);
      }
@@ -763,17 +748,14 @@

  // reset but remember selection first
  var select_before = this.selection.join(',');
  this.selection = new Array();
  this.selection = [];
  
  for (var n in this.rows)
  {
    if (!filter || (this.rows[n] && this.rows[n][filter] == true))
    {
  for (var n in this.rows) {
    if (!filter || (this.rows[n] && this.rows[n][filter] == true)) {
      this.last_selected = n;
      this.highlight_row(n, true);
    }
    else if (this.rows[n])
    {
    else if (this.rows[n]) {
      $(this.rows[n].obj).removeClass('selected').removeClass('unfocused');
    }
  }
@@ -820,8 +802,7 @@
  var num_select = this.selection.length;

  // one row
  if (id)
    {
  if (id) {
    for (var n=0; n<this.selection.length; n++)
      if (this.selection[n] == id) {
        this.selection.splice(n,1);
@@ -829,14 +810,13 @@
      }
    }
  // all rows
  else
    {
  else {
    for (var n=0; n<this.selection.length; n++)
      if (this.rows[this.selection[n]]) {
        $(this.rows[this.selection[n]].obj).removeClass('selected').removeClass('unfocused');
        }
    
    this.selection = new Array();
    this.selection = [];
    }

  if (num_select && !this.selection.length)
@@ -870,25 +850,20 @@
 */
highlight_row: function(id, multiple)
{
  if (this.rows[id] && !multiple)
  {
    if (this.selection.length > 1 || !this.in_selection(id))
    {
  if (this.rows[id] && !multiple) {
    if (this.selection.length > 1 || !this.in_selection(id)) {
      this.clear_selection();
      this.selection[0] = id;
      $(this.rows[id].obj).addClass('selected');
    }
  }
  else if (this.rows[id])
  {
    if (!this.in_selection(id))  // select row
    {
  else if (this.rows[id]) {
    if (!this.in_selection(id)) { // select row
      this.selection[this.selection.length] = id;
      $(this.rows[id].obj).addClass('selected');
    }
    else  // unselect row
    {
      var p = jQuery.inArray(id, this.selection);
    else { // unselect row
      var p = $.inArray(id, this.selection);
      var a_pre = this.selection.slice(0, p);
      var a_post = this.selection.slice(p+1, this.selection.length);
      this.selection = a_pre.concat(a_post);
@@ -909,8 +884,7 @@
  var keyCode = rcube_event.get_keycode(e);
  var mod_key = rcube_event.get_modifier(e);

  switch (keyCode)
  {
  switch (keyCode) {
    case 40:
    case 38: 
    case 63233: // "down", in safari keypress
@@ -951,8 +925,7 @@
 */
key_down: function(e)
{
  switch (rcube_event.get_keycode(e))
  {
  switch (rcube_event.get_keycode(e)) {
    case 27:
      if (this.drag_active)
   return this.drag_mouse_up(e);
@@ -988,8 +961,7 @@
  else if (keyCode == 38 || keyCode == 63232) // up arrow key pressed
    new_row = this.get_prev_row();

  if (new_row)
  {
  if (new_row) {
    this.select_row(new_row.uid, mod_key, true);
    this.scrollto(new_row.uid);
  }
@@ -1034,8 +1006,7 @@
scrollto: function(id)
{
  var row = this.rows[id].obj;
  if (row && this.frame)
  {
  if (row && this.frame) {
    var scroll_to = Number(row.offsetTop);

    // expand thread if target row is hidden (collapsed)
@@ -1058,8 +1029,7 @@
 */
drag_mouse_move: function(e)
{
  if (this.drag_start)
  {
  if (this.drag_start) {
    // check mouse movement, of less than 3 pixels, don't start dragging
    var m = rcube_event.get_mouse_pos(e);

@@ -1081,27 +1051,22 @@
    // get subjects of selected messages
    var names = '';
    var c, i, subject, obj;
    for(var n=0; n<this.selection.length; n++)
    {
      if (n>12)  // only show 12 lines
      {
    for(var n=0; n<this.selection.length; n++) {
      // only show 12 lines
      if (n>12) {
        names += '...';
        break;
      }

      if (obj = this.rows[this.selection[n]].obj)
      {
      if (obj = this.rows[this.selection[n]].obj) {
        subject = '';

        for (c=0, i=0; i<obj.childNodes.length; i++)
        {
     if (obj.childNodes[i].nodeName == 'TD')
          {
        for (c=0, i=0; i<obj.childNodes.length; i++) {
         if (obj.childNodes[i].nodeName == 'TD') {
            if (n == 0)
         this.drag_start_pos = $(obj.childNodes[i]).offset();

       if (this.subject_col < 0 || (this.subject_col >= 0 && this.subject_col == c))
       {
           if (this.subject_col < 0 || (this.subject_col >= 0 && this.subject_col == c)) {
         var node, tmp_node, nodes = obj.childNodes[i].childNodes;
         // find text node
         for (m=0; m<nodes.length; m++) {
@@ -1132,8 +1097,7 @@
    this.triggerEvent('dragstart');
  }

  if (this.drag_active && this.draglayer)
  {
  if (this.drag_active && this.draglayer) {
    var pos = rcube_event.get_mouse_pos(e);
    this.draglayer.css({ left:(pos.x+20)+'px', top:(pos.y-5 + (bw.ie ? document.documentElement.scrollTop : 0))+'px' });
    this.triggerEvent('dragmove', e?e:window.event);